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

Event and background colors and shading problem #325

Closed
bushrang3r opened this issue Sep 14, 2019 · 6 comments
Closed

Event and background colors and shading problem #325

bushrang3r opened this issue Sep 14, 2019 · 6 comments

Comments

@bushrang3r
Copy link

bushrang3r commented Sep 14, 2019

There is only one setting for event text shading but different color options for event background, past event background and future background.
This limits choices for background coloursimage_2019-09-15_07-48

Perhaps some sub-menus in settings and more colour options would solve this.

e.g.

Settings | Colors | Past

Day header text shading
Event text shading
Event background color

Settings | Colors | Today

as above

Settings | Colors | Future

as above

I agree there are already a lot of settings so perhaps there is an easier way to implement this....
Maybe day header text shading and event text shading could be just one option. ie. they are the same shading. This would create only 2 color options in the settings structure above, rather than 3.

@yvolk
Copy link
Collaborator

yvolk commented Sep 15, 2019

@bushrang3r Thank you for suggestions.

  1. Regarding settings structuring and ordering.
    As you may have noticed, I changed an order of settings in each section (mostly) to correspond to the order of elements on the screen: starting from Widget header and down...
    Similarly we do have "Past events" and "Current and future events" sections inside "Event filters". So I think that structuring Colors settings into "Past events", "Today's events" and "Future events" will be logical and will really help to find needed settings.

  2. Regarding day header shading vs event shading.
    I separated them only recently in Ability to change day header colour & event details colour #218. Previously Day header color was dependent on event color, but as some users noticed, it was always light and never black...
    Now we can set all widget text black or white thus adjusting for different backgrounds.

  3. And finally: Why do we have "Text shading" with only 4 values and not continuous "Text color" with the same color picker as for backgrounds?
    Actually "shading" means changing a "style", i.e. a group of attributes, together, including switching between white and black icons. See the four "Shadings" here: https://github.com/andstatus/todoagenda/blob/master/app/src/main/res/values/themes.xml

@bushrang3r
Copy link
Author

OK, thanks. I understand.

  1. The problem with this (as you can see in my screenshot) is that if my header text shading or event text shading is dark or black, I can't have any dark backgrounds for past, today, or future. They must all be light colours.

@yvolk
Copy link
Collaborator

yvolk commented Sep 15, 2019

@bushrang3r Yes, I see. So we need 2 * 3 text shading settings instead of 2.
I will do this.

yvolk added a commit to andstatus/todoagenda that referenced this issue Sep 15, 2019
yvolk added a commit to andstatus/todoagenda that referenced this issue Sep 15, 2019
…ure Day Headers and (separately) for Events
@yvolk
Copy link
Collaborator

yvolk commented Sep 16, 2019

Done in v.3.0.5 Please check in #320

In order to minimize number of settings the app hides color settings that are not actual. It hides color settings for Day headers if "Show Day headers" setting is turned off. "Past events" section is removed completely if no past events may be shown and no task sources selected.

yvolk added a commit to andstatus/todoagenda that referenced this issue Sep 16, 2019
@yvolk yvolk closed this as completed Sep 17, 2019
@yvolk
Copy link
Collaborator

yvolk commented Sep 20, 2019

@bushrang3r I like this extention so much that I even started to consider changing default today's colors. Previously all events where white text on dark background.
Now we can empathize today by inverting the colors. I found that such a change really makes today easily visible.
What do you think?
Screenshot_2019-09-20-08-19-03-570

@bushrang3r
Copy link
Author

@yvolk
Yes! It's wonderful! I love it. It's great to be able to show past, today, and future with different colours. Very easy and quick to differentiate events and days. It's very customizable now. I'll keep testing this beta and keep using it everyday.
Thanks for all your hard work!

yvolk added a commit to andstatus/todoagenda that referenced this issue Sep 22, 2019
…be better highlighted. Now it's black on light background
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

No branches or pull requests

2 participants