Skip to content
This repository has been archived by the owner on May 14, 2020. It is now read-only.

Slightly darken fonts for light theme to improve contrast #15

Merged
merged 1 commit into from
Jan 10, 2018

Conversation

fossfreedom
Copy link
Collaborator

The current font colour for the light theme does not
provide enough of a contrast to be easily legible. This
commit reduces the contrast colour by approx 7%
This resolves for #406

ubuntu budgie - new development artful running - oracle vm virtualbox_008

compared to current

ubuntu budgie - new development artful running - oracle vm virtualbox_005

The current font colour for the light theme does not
provide enough of a contrast to be easily legible.  This
commit reduces the contrast colour by approx 7%
This resolves for #406
@jeremy447
Copy link

I can't test for now but it seem much much better on the screenshot. That said I wonder if it's enough ?
The full dark screenshot seem even better.

@fossfreedom
Copy link
Collaborator Author

The black font breaks the aesthetic looks of Arc. I'm trying to ensure we work within the Arc parameters. I would suggest this needs testing to confirm its usability. To my eye it works quite nicely.

@fossfreedom
Copy link
Collaborator Author

@NicoHood can we merge the current two PRs?

@NicoHood
Copy link
Collaborator

NicoHood commented Jan 2, 2018

@fossfreedom I did not test them yet, I can try if my internet is fast enough next week. From just looking at the changes and screenshots it is a good idea, of course. If you think the changes are all good, merge it. If you explicitly wish me to test it first, I will do.

Oh and by the way: great job from your side! The changes so far are every useful, I am really happy arc is still alive.

@fossfreedom
Copy link
Collaborator Author

I would prefer to have positive feedback first before making the more obvious changes like these. Next week would be fine.

BTW - please can we aim for a formal signed release in the middle of Jan? This will then allow me to push to Debian & Ubuntu - specifically I am aiming for Alpha 2 of Ubuntu Budgie 18.04 Feb 1st.

@NicoHood
Copy link
Collaborator

NicoHood commented Jan 2, 2018

@fossfreedom okay, I will test and give feedback soon. Sure I can, just remind me or give me a deadline. Those two PRs should be included of course.

@fossfreedom
Copy link
Collaborator Author

COP (close of play) Fri 19th Jan. Cheers.

@NicoHood
Copy link
Collaborator

NicoHood commented Jan 6, 2018

Can you please give me some advice how to use the new sass? You made some changes, right? Gulp is not required anymore? You could add this to:
https://github.com/NicoHood/arc-theme/blob/master/HACKING.md

It would be nice if we can automatically generate the css files within the makefile. We can then decide if we still want to commit them or not.

@NicoHood
Copy link
Collaborator

NicoHood commented Jan 6, 2018

I've tested the changes and do not see any problem. I am however normally using arc-darker with global dark theme enabled. This change should be fine.

@NicoHood NicoHood added this to the 20180114 milestone Jan 8, 2018
@NicoHood
Copy link
Collaborator

NicoHood commented Jan 8, 2018

I found out that the darker colors are not applied to arc-darker:
screenshot from 2018-01-08 20-13-49
screenshot from 2018-01-08 20-14-37

@NicoHood NicoHood mentioned this pull request Jan 8, 2018
@fossfreedom
Copy link
Collaborator Author

fossfreedom commented Jan 8, 2018 via email

@NicoHood
Copy link
Collaborator

NicoHood commented Jan 9, 2018

Its the xfce4 setting menu. I noticed this by accident.

@fossfreedom
Copy link
Collaborator Author

@NicoHood I'm not an XFCE user so I can't really say if the following change works in general so please can you test?

It appears that that appearance window is a GTK-2 window - so gtk-2 stuff is in common/gtk-2.0

so the text color/fg color between the arc (gtkrc) arc-darker (gtkrc-darker) and arc-dark (gtkrc-dark) are different deliberately at the moment.

ubuntu budgie - new development artful running - oracle vm virtualbox_030

if you change the fields shown in the picture to be the same as Arc (gtkrc) then the text colour is the same - if you feel this looks better, push a commit on that file

@NicoHood NicoHood merged commit 5680ad1 into master Jan 10, 2018
@NicoHood NicoHood deleted the darkenfonts branch January 10, 2018 18:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants