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

set text color for menus (define global) #67

Closed
wants to merge 1 commit into from

Conversation

bekuno
Copy link
Member

@bekuno bekuno commented Mar 7, 2020

fix #66

  • remove theme definitions for api 11 an 14, merge them in the default configuration files (we are on min sdk 16)
  • add text color definitions
  • define the default text color (white) fo the AppTheme

fix cgeo#66

- remove theme definitions for api 11 an 14, merge them in the default configuration files (we are on min sdk 16)
- add text color definitions
- define the default text color (white) fo the AppTheme
@bekuno bekuno self-assigned this Mar 7, 2020
@bekuno bekuno requested a review from Lineflyer March 7, 2020 12:59
@bekuno
Copy link
Member Author

bekuno commented Mar 7, 2020

@Lineflyer Can you test this carefully on devices with different Android versions?

@bekuno bekuno added the Do not merge / WIP Not for merging and/or work still in progress label Mar 7, 2020
@bekuno
Copy link
Member Author

bekuno commented Mar 8, 2020

This change has global side effects.
The formatting needs to be completely recreated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do not merge / WIP Not for merging and/or work still in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant