Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Color issues #5040

Closed
26 tasks done
sampaiodiego opened this issue Nov 24, 2016 · 12 comments · Fixed by #5089 or #5180
Closed
26 tasks done

Color issues #5040

sampaiodiego opened this issue Nov 24, 2016 · 12 comments · Fixed by #5089 or #5180
Assignees
Milestone

Comments

@sampaiodiego
Copy link
Member

sampaiodiego commented Nov 24, 2016

Your Rocket.Chat version: 0.47.0-develop

After the merge of #4803 I've found many color issues, so I'll try to gather them all here.

  • Missing borders on My Account page: image
  • Date divider white: image
  • Favorite icon without gray border: image
  • Message text box without border: image
  • white and transparent "unread bar": image
  • Room's list hover too white: image
  • Livechat menu has a lighter background: image
  • "unread messages" bar at the very top disappears whenever the scroll event is triggered and completed (thanks @frdmn - post): image
  • It used to have a line above the "unread messages" divider: image
  • Code styles: image
  • the background of what before was a grayish-white is now full-white: image
  • Attachment buttons darker then before: image
  • @all changed to red; @mention changed to a darker blue; darker role tag background: image
  • Other's @mention changed to gray: image
  • Message action buttons styles: image
  • emoji picker search box background used to be white: image
  • message editing highlight got red and the text field got white: image
  • Expand/collapse button on admin are got white (gray only on hover): image
  • File upload got a more transparent background and it's not perfectly aligned: image
  • "user is typing" is breaking line if you're editing some message: image
  • messages scrollbar got white: image
  • Login form auto-fill sets a background on text fields (thanks @MartinSchoeler - post): image
  • Background no changing on hovering attachment buttons: image
  • Server connection status: image
  • light blue buttons got white: image
  • First lines of a multi-line message increases the size down: image
@frdmn
Copy link
Contributor

frdmn commented Nov 24, 2016

Also @TheReal1604 told me that the loading spinners (pulsing dots) are gone, which I can confirm.

I guess that's related to that merge as well.

@frdmn
Copy link
Contributor

frdmn commented Nov 24, 2016

And the "unread messages" bar at the very top disappears whenever the scroll event is triggered and completed:

test

@engelgabriel
Copy link
Member

Initial fix via 82847da

@MartinSchoeler
Copy link
Contributor

This happens if you use the auto complete in the login page
screenshot from 2016-11-28 09-41-01
screenshot from 2016-11-28 09-39-56

@engelgabriel engelgabriel added this to the 0.47.0 milestone Nov 28, 2016
@timkinnane
Copy link
Contributor

Thanks @sampaiodiego for finding all those issues. I knew about some of them, but thought the effort to get a more consistent baseline was a net improvement, even with some bugs. My PR had to survive merges with multiple versions too, so I think some of those issues are the result of conflicts being resolved without noticing all the details they impact. Hopefully I'll get some time to help resolve the remaining ones.

@karlprieb
Copy link
Contributor

@timkinnane if you get some time call me on Rocket.Chat :) I'm fixing these errors.

@timkinnane
Copy link
Contributor

timkinnane commented Nov 30, 2016

@karlprieb nice work, thanks. I find it hard to match up with you guys time zone for calls outside work hours, but will try and catch you online in demo.rc.
My main concern is just that the color scheme variables are used to resolve any issues. Not worried if the colors themselves need changing or we use one instead of another. Just wanted to avoid any hard coded colors in the less files, to achieve the main objective of letting people set their own color scheme without having fixed colors they can't change ruining the application of their scheme.

@Sing-Li
Copy link
Member

Sing-Li commented Dec 8, 2016

More unintended CSS changes due to the PR continues to pour in.

Not all impact has been addressed. Re-opening this case.

@Sing-Li Sing-Li reopened this Dec 8, 2016
@karlprieb
Copy link
Contributor

I will look these changes.

@IQ2022
Copy link

IQ2022 commented Dec 8, 2016

Unread messages stays longer than it should, also it has wrong place it overlays the username, so if you want to change or react to a msg, it wont allow you to click on the setting wheel..

image

@engelgabriel engelgabriel modified the milestones: 0.48.0, 0.47.0 Dec 9, 2016
@engelgabriel
Copy link
Member

image

@engelgabriel
Copy link
Member

@IQ2022 can you open a new issue for the "Unread messages stays longer than it should, also it has wrong place it overlays the username"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

8 participants