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

Testing PR: reduce the orange in selected rows #1303

Merged
merged 6 commits into from Apr 29, 2019
Merged

Conversation

Feichtmeier
Copy link
Member

@Feichtmeier Feichtmeier commented Apr 18, 2019

Edit: updated with mads correction

Screenshot from 2019-04-19 16-03-24
@madsrh @ubuntujaggers @clobrano @mpt 😇

Why?

  1. We love our orange but it is a problematic color - especially when it is used as a big surface like the rows. Staring at it every day using ubuntu as my daily driver on my private PC, it becomes really hard for the eyes.

  2. The full color selection is a little bit dated, when you comepare it to...
    Spotify
    photo_2019-04-18_17-34-41
    Vscode
    photo_2019-04-18_17-34-48
    Windows10 settings
    image
    Gmail on tablets
    image
    Battle.net Client
    image
    Imgur settings
    image
    discourse settings
    image
    gnome gitlab
    image

  3. it "fits" somehow to firefox tabs design
    image
    and To our gnome shell panel button selection
    image
    and to our gnome shell workspace selections
    image

@didrocks
Copy link
Member

Does it impact the firefox theme as well? I guess the change should be everywhere then (firefox, libreoffice and such, if they don't use the default value). I would have that reviewed by mpt as well, having no strong opinion myself, but I liked personnally that bit of branding (I think we found the right balance) :)

@Feichtmeier
Copy link
Member Author

Feichtmeier commented Apr 18, 2019

It is just selected rows in GtkListBoxes
Treeviews are not affected, but those are legacy ui elements and not used very often in modern apps
Also treeview rows are not as fat as listbox rows. Thus the orange surface is not THAT big.

@madsrh
Copy link
Member

madsrh commented Apr 19, 2019

...but I liked personnally that bit of branding (I think we found the right balance)...

+1

The full color selection is a little bit dated, when you comepare it to...

I agree. As you know, we tried this style for at while earlier and it didn't grow on us. Many months later I'm still not in love, but I see your very valid point. Personally I would miss the solid orange, BUT if we keep this suggested style/design I think there's too many indicators:

  • Orange line
  • Bold font
  • Gray background

IMHO that is too much. The bold font might be hardcoded (in Nautilus). Anyway, with the current PR I find the gray background too dark (if it should be there at all):

mockup1

OR play with the text color of the sidebar items that isn't selected, like this:

mockup1b

@clobrano
Copy link
Member

+1 to what Didrocks and Mads said.
About the improvements, I think that using only the orange border and some bold text is not enough to immediately see the selection, and a background color is necessary.
It's a about a week now that I'm using a gray highlights in place of the orange one, it's not bad and probably the tone can be improved, but in any case I don't feel sold to this change and it doesn't look like a better solution

@Feichtmeier
Copy link
Member Author

Feichtmeier commented Apr 19, 2019

Nope the bold font is not hardcoded.
Lighter gray and normal font is doable. I think when looking at the other software screenshots, it is a pretty heavy contrast to our current solution.
The orange is def. too strong to use it on the full row. I'll try @madsrh 's idea and post it

I think that using only the orange border and some bold text is not enough to immediately see the selection

I have 0 problems to see the selection in the various software I posted, which I actually all use on my computer, apart from gmail for tablets and windows 10 (ofc ;D)
Whereas staring at the super bright and intense orange bar is very problematic, and when you look at some reviews of the theme the orange is always the one of rather negative points (if there are any negative points)
This intense orange works if it is an accent color. As with stripes or dots

@Feichtmeier
Copy link
Member Author

Screenshot from 2019-04-19 16-03-24
@madsrh that's without the bold font, a bit darker fg_color and a bit lighter fg_color for normal rows and a lighter gray

lighter gray bg, normal font, darker fg_color, lighter fg_color for not selected rows
@madsrh
Copy link
Member

madsrh commented Apr 19, 2019

...that's without the bold font, a bit darker fg_color and a bit lighter fg_color for normal rows and a lighter gray

IMO the gray should be much lighter.

...I think that using only the orange border and some bold text is not enough to immediately see the selection, and a background color is necessary.

I see your point, but as @Feichtmeier illustrated above that is the design trend 🤷‍♂️Our current orange solution is perfect in terms of accessibility and maybe less could work. I know @clobrano will strongly disagree but after looking at this for a while, I'm starting to like this one 🙈 (if we can't keep the current).
My problem with the initial suggestion in this PR is that is doesn't look very slick and IMVHO this one does (if the user can spot it 😆 ). This is just based on the screenshots/mockups here, I havn't tried anything in the real though.

@Feichtmeier
Copy link
Member Author

I'll try the one you suggested!

@clobrano
Copy link
Member

I see your point, but as @Feichtmeier illustrated above that is the design trend

I see, but just do not sacrifice usability to trends.

Selected bookmark is harder to spot at a first glance, which is a -1 for me, and even more if the only reason to change this is a personal view about the orange being "too much" in this specific use case.

if we can't keep the current

I hope the current is still an option, though :)

I also think that all rows should keep the same hightlight style.

Moreover, from my point of view, this is a sofistication that doesn't get along with the rest of Nautilus' style, which is nice for it's simplicity.

About the other use cases: all of them uses both border and text color/bold and if not, they highlight the full row anyway.

I think it is not by chance that other OSes very user centric like OSX and Elementary cdo not ship with this kind of sidebar's row selection style.

@Feichtmeier
Copy link
Member Author

The problem is the colour, not the filled bg styling
Google's material design 2.0 uses a filled bg, too but they do not use such a hardcore intense colour like we do
image
image
Also the outlook screenshot you posted shows that it works with a different color saturation and/or reduced colour brightness. Sadly this would mean salmon again in our case ;)
If it would be transparentized colours like in material design 2.0 it would also be an option. But it is this super intense orange.

@Feichtmeier
Copy link
Member Author

Feichtmeier commented Apr 20, 2019

@madsrh I hope this fits to your fav:
image
hover
image
active
image

@clobrano
Copy link
Member

The problem is the colour, not the filled bg styling

I don't agree, but I understood what the problem is for you, however we're testing a half backed solution respect the one listed above, this is what I was saying

@Feichtmeier
Copy link
Member Author

Why is this solution half backed, because you don't like it? It works in different apps and it works in nautilus. Also, could we please stop with this usability argument? You can not seriously tell me that you can not 100% say what the selected row is.

@clobrano
Copy link
Member

Why is this solution half backed, because you don't like it?

Nope, because respect the other example there's only the colored border and not the background color, nor colored/bold text.

You can not seriously tell me that you can not 100% say what the selected row is.

That's what I meant by usability. Of course I can "see" what's selected, but it's not easy to spot as before

No need to be aggressive though, I am only speaking my voice since I don't see the need for this change. However, if the rest of the team agrees, as I always did, I won't stop this PR.

@Feichtmeier
Copy link
Member Author

It's still a testing pr and I tried to implement mads mock-up. I'm not being aggressive I'm throwing back the ball of your passively aggressive attempts to wreck through three obvious problems with this full coloured bright orange bar with the usability argument. Ofc usability matters. And many times we dropped an idea because usability would have been lost. In this case it's not lost it's maybe reduced by a small amount. It's not like the stripe is the only solution possible. It's just that the 1) bright orange 2) full colour solution is bad.
I'm just waiting for something different to pop up though I would already prefer the transparent BG + stripe or gray BG + stripe over this orange bar. In ambiance the orange was way less intense btw.

@clobrano
Copy link
Member

'm not being aggressive I'm throwing back the ball of your passively aggressive attempts to wreck through three obvious problems with this full coloured bright orange bar with the usability argument.

I think you misunderstood and this is quite unfair. Usability is reduced too much from my point of view, that's all I wanted to say. Of course I am discussing a work in progress, if it was already fixed there won't be no point in discussing. But anyway, I think I already replied enough, for the current state of the work my idea is written above, I'll try to figure out something that can please anybody and if I can't peace.

@Feichtmeier
Copy link
Member Author

Alright, just push it in this branch. I pause here until then, or just tell me that you want to change and I'll do it

@ubuntujaggers
Copy link
Contributor

"Of course I can 'see' what's selected, but it's not easy to spot as before"

For me it's about peripheral vision, and knowing where to look before I even look.

The advantage of the full coloured or shaded background is it draws your eye and you immediately know where to glance. In screenshots that don't have it, it's obvious when you're looking directly, but my eyes jump up/down the side bar to find it. It's a miniscule difference, but there's a split second when my eyes are "hunting" for the visual clue, whereas you can't miss it when it's a big stripe :)

Also the orange "stub" is more eye-catching when the window is in front of another window, because then it's surrounded by white. I find it jumps out less when the window is on top of gaily coloured wallpaper. I think stub highlights with no other clue work better for apps that are normally full screen (e.g. mobile apps) because the edge of the screen already draws the eye and the coloured accent is usually next to black or white.

This doesn't mean I'm opposed to the orange stub at all, but I probably prefer the versions that combine it with a neutral fill across the full width (which some of the style exemplars have) and which looks fairly smart IMO.

@ubuntujaggers
Copy link
Contributor

Looking at the examples, when there's no fill, the text is more strikingly different for the selected item. Either through having a different colour:

image

...or through a more pronounced difference in boldness:

image

It might work to copy the style of the Ubuntu website and make unselected items a lot fainter?

image

@madsrh
Copy link
Member

madsrh commented Apr 22, 2019

It might work to copy the style of the Ubuntu website and make unselected items a lot fainter?

Yes, I was aiming for something similar here but I think if the unselected items are too faint, it sacrifices the readability and makes it look disabled.

I hope this fits to your fav

It does @Feichtmeier 😄 It's perfect in the sense that it reduces the orange in selected rows but I still love the current stub, so I won't "vote" for either ⚖️

@CDrummond
Copy link
Contributor

@ubuntujaggers the problem with the lighter text is that this is often used to indicate that an item is disabled. I have the same issue with inactive tabs - where also a lighter text colour is used (making them look disabled). Active tabs have a bold font and border, I can't see the point of the ligher text for inactive ones as well. So, even here, a bolder font for the active item, and standard for in-active (but not disabled) would be better (IMVHO). Also, using a gray sripe would match the active menu item.

@Feichtmeier
Copy link
Member Author

Variations in comparison:

  1. selected: stripe, no bg, normal font weight, darker text | unselected: no stripe, no bg, lighter text
    Screenshot from 2019-04-24 10-54-01
    Screenshot from 2019-04-24 10-53-28

  2. selected: stripe, gray bg, normal font weight, darker text | unselected: no stripe, no bg, lighter text
    Screenshot from 2019-04-24 11-00-35
    Screenshot from 2019-04-24 11-03-48

  3. selected: stripe, gray bg, bold font weight, darker text | unselected: no stripe, no bg, normal text color
    Screenshot from 2019-04-24 11-09-03
    Screenshot from 2019-04-24 11-08-55

  4. selected: no stripe, gray bg, normal font weight, darker text, | unselected: no stripe, no bg, normal text
    Screenshot from 2019-04-24 11-18-53
    Screenshot from 2019-04-24 11-18-43

  5. bonus...:
    selected: no stripe, gray gradient, normal font weight, dark text | unselected: no stripe, no bg, normal text color
    Screenshot from 2019-04-24 11-17-24
    Screenshot from 2019-04-24 11-17-14

@madsrh
Copy link
Member

madsrh commented Apr 24, 2019

Always a good idea with a poll 😂
I like 3, but with an even lighter gray

@Feichtmeier
Copy link
Member Author

Haha I just wanted to sum it up :p
I think I'm also liking 3 the most, closely followed by 4
The grey could indeed be lighter

@ubuntujaggers
Copy link
Contributor

Always a good idea with a poll joy
I like 3, but with an even lighter gray

+1.

For number 1, my "old man" effect :) is even more pronounced for dark theme. The way I test it is to look at the files area rather than the sidebar, to see how obvious it is (out of the corner of my eye) where the sidebar selection is.

@Feichtmeier
Copy link
Member Author

Feichtmeier commented Apr 24, 2019

I don't want to pretend to be the smart-ass, but to see where you are, one should rather look on the pathbar! Because that is the widget that shows where you are, no matter how far you navigated.
The Sidebar is rather for "bookmark" clicking. There is a bunch of default bookmarks you can click on on and you can also add more with Ctrl+D. When you navigated 1 level further in your bookmark it is already not :selected anymore:

image

Then press Crl+D to bookmark it and THEN nautilus notices that you are inside a bookmarked location again and displays the selection
image

@clobrano
Copy link
Member

I don't want to pretend to be the smart-ass, but to see where you are, one should rather look on the pathbar! Because that is the widget that shows where you are, no matter how far you navigated.

This is true, but it's like to test the breaking system looking whether the speedometer goes down :D
I'll double check later because I'm afk, but n3 on dark variant seems barely visible

@Feichtmeier
Copy link
Member Author

Feichtmeier commented Apr 24, 2019

I used a transparentized silk now for the dark theme, how about this @clobrano ?

image
image

And a slightly more transparentized black for the light theme, how about this @madsrh ?
image

@clobrano
Copy link
Member

I'm sorry, I should've been more specific. I meant that the orange border is barely visible, the background color is OK, and in effect this is the same in all the variants that use this border IMO.

@Feichtmeier
Copy link
Member Author

Feichtmeier commented Apr 24, 2019

It should be just an accent, and by this definition it is visible "enough" even if this might be subjective again...
The orange blob looks even more awkward in the dark theme tbh. The orange is almost burning through the LCD

@madsrh
Copy link
Member

madsrh commented Apr 24, 2019

And a slightly more transparentized black for the light theme, how about this @madsrh ?

👍

@Paz-it
Copy link

Paz-it commented Apr 25, 2019

+1 with @didrocks and @clobrano arguments.
Personally, I grow to like the current orange and don't see reason to change it.
I took the time to make sure how I feel before I said it, though...

@Feichtmeier
Copy link
Member Author

Okay, that are pretty many -1 and some neutrals so I'd say let's skip it again? :)

@Feichtmeier
Copy link
Member Author

don't see reason to change it.

In the opening post/pr description are three good reasons to change it.
If that still didn't convince you, maybe this real life picture of my xps screen does:

IMG_20190428_205047_379

The orange bar is almost burning through the screen. I'd say it looks horrible and that this is a very obvious improvement

@Feichtmeier
Copy link
Member Author

After agreement on telegram discussions I merge the gray selection without the orange stripe to master now to test it again for some months before "the hot 19.10 phase"

@Paz-it
Copy link

Paz-it commented Apr 30, 2019

I'd say it looks horrible

I'd say : What a noisy horrible wallpaper you have there 😄

@eaglersdeveloper
Copy link
Contributor

eaglersdeveloper commented May 1, 2019

telegram discussions

[humor]
You have Telegram?! I don't have access to Telegram from Russia 😬😬😬
[/humor]

@Feichtmeier
Copy link
Member Author

No!? That's sad :(

We are not really satisfied with the gray row and might come up with something different soon

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 this pull request may close these issues.

None yet

8 participants