TBog / TBLauncher

based on https://github.com/Neamar/KISS
GNU General Public License v3.0
137 stars 18 forks source link

Change the launcher's (too) bright green colors #72

Open ltguillaume opened 3 years ago

ltguillaume commented 3 years ago

The main app's toxic green gives it a bit of an old terminal/XT computer vibe. I'm all for retro, but it does mean it doesn't blend with the rest of the OS. Therefore, it would be fantastic if the app would either 1) follow the accent color set in Android's Settings -> Display -> Styles & Wallpapers, or 2) allow to set the accent color and the background color for TBLauncher's menus and settings manually.

I guess the next step would be to support dark mode when set in Android.

As I understand, the light green background is partly there because there's no border around the set color's preview (so they could disappear when e.g. white is selected). Perhaps the following can be used to easily set a border around the color preview: https://stackoverflow.com/a/3264140

TBog commented 3 years ago

I don't know how to find the color you set in Settings -> Display -> Styles & Wallpapers. Also I'm not sure what you're talking about, I don't have any menu like that on my Samsung phone and the Android 10 emulator doesn't have a color option, only Dark mode

The green you speak of blends with my phone perfectly (Samsung has OS themes), but I think I understand what you want. From what I understand you want an option to change the green color of the popups and dialogs

  1. popup text color: image

  2. dialog border: image

Is that all you need when you say (too) bright green colors?

ltguillaume commented 3 years ago

I don't know how to find the color you set in Settings -> Display -> Styles & Wallpapers.

Librechair does it like this: https://github.com/linsui/librechair/blob/c999d70cf2a57fb7243159748f23b49482a6d8d7/lawnchair/src/ch/deletescape/lawnchair/colors/colorResolvers.kt#L54

Also I'm not sure what you're talking about, I don't have any menu like that on my Samsung phone and the Android 10 emulator doesn't have a color option, only Dark mode

You're right, sorry, it was put behind the Developer options later in development, apparently it was LineageOS that put it where I said it was. But it's still available on any Android 10 device, see https://www.androidauthority.com/change-system-accent-color-android-10-1025163/

The green you speak of blends with my phone perfectly (Samsung has OS themes), but I think I understand what you want. From what I understand you want an option to change the green color of the popups and dialogs

1. popup text color:
   ![image](https://user-images.githubusercontent.com/6465804/103245037-8c587a80-4967-11eb-8570-724ddd306e7c.png)

2. dialog border:
   ![image](https://user-images.githubusercontent.com/6465804/103245005-6af78e80-4967-11eb-8e56-7d2020075423.png)

Is that all you need when you say (too) bright green colors?

Those colors 1 and 2 indeed, but also 3 the Settings background/accent colors and 4 the highlighted (ripple) color of #20.

Simply put, I'd like to get something like a neutral grey color (such as #eeeeee) to be the overall accent color where the green is now.

TBog commented 3 years ago

The ripple color has its own issue for now.

I don't think a color chooser for the color chooser (Settings activity) is needed.

I'll change the default Settings background to black/dark and the text to green (now that I figured out how to overwrite the generated preference menu). The Settings activity generates the layout from an XML file and I can only set the theme of the activity. It's not possible AFAIK to make/change a theme programmatically and I don't want to change the way the preferences are currently handled.

I'll think of a way to use the colors from the result list preferences to also change the settings. For example use the "Result list - background color" as the Settings background and the "Result list - text color" for the text, but some combinations may not work or be too hard to read and this would block the user from changing them back.

What do you think?

ltguillaume commented 3 years ago
  1. I'll think of a way to use the colors from the result list preferences to also change the settings. For example use the "Result list - background color" as the Settings background and the "Result list - text color" for the text.

That would make more sense, as it would allow customization.

For what it's worth, this is my view on the current style of the app: I think the green color isn't neutral (i.e. it doesn't fit the OS), and a color like it by default may deter people from starting to use TBLauncher in the first place, which would be a pity. If it's customizable, that's a great step. And I suggested using the system's accent color for if you want to make a better first impression towards new users, as it would integrate better into their system.

TBog commented 3 years ago

I'll add the system's accent color as a default color for the borders.

But I'm not sure the amount of work needed to change the colors of the Settings activity is warranted. What if the user chooses a combination of colors that is too hard to read? This would block the user from changing them back.

ltguillaume commented 3 years ago

Ah that could be the case, I don't know about the details of customizing the theme programmatically. But if the Preferences Activity is just based on a listView, I guess you could simply set the text color in (and setTint() for drawables like a color preview border) inside getView().

TBog commented 3 years ago

Ah that could be the case, I don't know about the details of customizing the theme programmatically. But if it's just a listView, I guess you could simply set the text color in (and setTint() for drawables like a color preview border) inside getView().

All I do is setContentView(R.layout.activity_settings); in onCreate I have limited control of the generated views

TBog commented 3 years ago

Like I said, I'll research some more when I have time and other issues are less important. Right now the only fix I can think of for the Settings activity is to change the background to something dark or black.

ltguillaume commented 3 years ago

I've been searching for a solution, but the only thing I could find was to set up different themes for the different accent colors, so setting something like textColor and colorAccent programmatically seems impossible for PreferenceFragmentCompat. So that's definitely not worth it.

TBog commented 3 years ago

I'm starting to heavily dislike the theme api that is used by the Preferences. I'm still working on this. Would you be willing to help test a preview build? I'm not the best judge for chromatic changes.

TBog commented 3 years ago

There is a new release v5.4-colors.2 with

  1. settings activity themes
  2. customizable popup colors
  3. ripple color

Need some feedback and help choosing colors for the settings activity themes. Screenshot_1610649464

Here is a preview of the popup with yellow and red Screenshot_1610649390

ltguillaume commented 3 years ago

Great stuff! I'll check it out very soon!

ltguillaume commented 3 years ago

What I've seen so far:

  1. The changed highlight color is not applied to the button to the left and the one to the right "⋮" (settings popup) of the search input.
  2. Would be nice to have a color setting for the highlight color, bright green titles, the secondary text and the scroll bar in the settings, but that's obviously very low priority.
TBog commented 3 years ago
  1. The changed highlight color is not applied to the button to the left and the one to the right "⋮" (settings popup) of the search input.

Will fix.

  1. Would be nice to have a color setting for the highlight color, bright green titles, the secondary text and the scroll bar in the settings, but that's obviously very low priority.

Can't have customized colors for the Settings activity. What I can do is add more themes besides the 3 (Black, White, Dark) that are now. Please help with colors. I don't know what combination to use.

ltguillaume commented 3 years ago

Compared with the settings I have elsewhere, I'd have these colors for Settings:

Not sure if it looks great right away.

TBog commented 3 years ago

@ltGuillaume can you please look at the latest pre-release and give some feedback?

ltguillaume commented 3 years ago

(1) seems to be fixed! Nice!

I see no differences regarding (2), but I think that's OK.

  1. Also, the lines underneath headers in popups cut off the lower part of the title's characters (with the g in Settings and Widget).
TBog commented 3 years ago

I see no differences regarding (2), but I think that's OK.

You're talking about the scrollbar? Which one?

  1. Also, the lines underneath headers in popups cut off the lower part of the title's characters (with the g in Settings and Widget).

If I change the draw order and have the text over the bar would the overlap be ok?

ltguillaume commented 3 years ago

I see no differences regarding (2), but I think that's OK.

You're talking about the scrollbar? Which one?

No I meant that I don't know if you've changed anything to (the colors in) the Settings. I see no new color profile, as you suggested with "Please help with colors. I don't know what combination to use.", but that's okay.

  1. Also, the lines underneath headers in popups cut off the lower part of the title's characters (with the g in Settings and Widget).

If I change the draw order and have the text over the bar would the overlap be ok?

I think it would still look very messy tbh.

TBog commented 3 years ago

The White theme in the setting is monochrome now, as in only black, white and shades of gray. It took some time to figure out how to change every color and remove the green.

The settings theme also changes the appearance of the edit tags dialog and the custom icon dialog.

Next I'll make the black theme monochrome and maybe add others.

Could you also check the other dialog and popup windows?

ltguillaume commented 3 years ago

Aah yes, I tried all three for the first time with this version, whereas I only saw the dark and black ones with the previous. White looks nice though!

What I've seen so far:

  1. The popup when long-pressing an app has a scroll bar, which seem unnecessary. Moreover, this scroll bar goes "outside" of the popup due to the rounded corners. The colors are correct.
  2. The Rename App and Add Tag dialogs are still green (border, title and the buttons "cancel" and "rename"/"ok").
TBog commented 3 years ago

With the white theme the rename dialog should look like this SmartSelect_20210118-214549_TinyBit launcher.jpg

ltguillaume commented 3 years ago

Aah that makes sense. I set the theme to white and I'm seeing this: image I'd say the gradient doesn't do it much good, gives an early 2000s feel.

TBog commented 3 years ago

@ltGuillaume can we close this issue with the current state of Feature - menu color preferences (6)

ltguillaume commented 3 years ago

Fantastic! Thank you, great work!

TBog commented 3 years ago

Cursor color not changed. Possible fix: https://www.tfzx.net/article/2715806.html or: https://stackoverflow.com/questions/35583320/changing-androids-edittext-cursor-pointer-color-programmatically?noredirect=1&lq=1

ltguillaume commented 3 years ago

Cursor color not changed. Possible fix: https://www.tfzx.net/article/2715806.html or: https://stackoverflow.com/questions/35583320/changing-androids-edittext-cursor-pointer-color-programmatically?noredirect=1&lq=1

Apart from this, I just found out that the "Do you want to create a shortcut" dialog is still green.

TBog commented 3 years ago

I just found out that the "Do you want to create a shortcut" dialog is still green.

Yes, forgot about it, I'm mostly testing on API 24 and that dialog is not used. Thanks for the reminder.

ltguillaume commented 3 years ago

Aaah that makes sense, yes. Android 7.0 is still my testing ground as well, especially since the AVD is much cleaner and faster.

ltguillaume commented 3 years ago

I just realized that the cursor "teardrop" in the Rename App dialog isn't green, unlike in the search bar, although its transparency has been corrupted (there's a black square around it).

ltguillaume commented 3 years ago

I just realized that the cursor "teardrop" in the Rename App dialog isn't green, unlike in the search bar, although its transparency has been corrupted (there's a black square around it).

For https://github.com/TBog/TBLauncher/releases/tag/v5.4-modes.1

So, what remains to my knowledge is:

  1. "Do you want to create a shortcut" dialog is still green
  2. Cursor in search now is yellow (could be some standard Android color, as the default Android color to highlight a row in my DroidShows app is also this yellow).
TBog commented 3 years ago

For https://github.com/TBog/TBLauncher/releases/tag/v5.4-modes.1

  • Cursor "teardrop" color in search isn't green anymore, don't know which color it now uses (secondary text color perhaps?). However, the cursor seems to have changed to yellow since this build.

It's the color set for ripple in the results list. Should this be customized separately?

  • I think both the rename dialog and the change icon dialog now adhere to the Preferences theme, instead of the colors & corner radius setting you can set for popups, right? But maybe it still uses some of the (text) colors set by the user? Judging from the following screenshot, changing the settings theme for a moment to white has strange effects

I don't understand what is wrong.

The popup is customizable to a greater degree because it's generated from code and the user is using that very often, at least in my case.

The rename dialog, custom icon dialog, etc. are using the settings theme.

So, what remains to my knowledge is:

  1. "Do you want to create a shortcut" dialog is still green

Yes

  1. Cursor in search now is yellow (could be some standard Android color, as the default Android color to highlight a row in my DroidShows app is also this yellow).

Should be the result list ripple color, same color is used for the Search icons ripple and the text highlight. We could make them separate.

ltguillaume commented 3 years ago

1.

I don't understand what is wrong.

Since it's a popup, I would assume for the Rename and Change Icon dialogs that the popup colors and corner radius setting would be used. And the text is difficult to read on this light settings theme + shadow makes it looks like I'm drunk.

It was (more) like the popup before, and I'd say the current implementation is quite a step backwards in terms of consistency/unity within the app. That being said, I understand these dialogs are used less and therefore have less of a priority for customizibily. I think if the corner radius could be adjusted to the setting for popups, it could already be a step in the right direction.

2.

Should be the result list ripple color, same color is used for the Search icons ripple and the text highlight. We could make them separate.

I guess separating them would make sens, since they're "borrowed" from another element then.

But here's a screenshot while the results list touch color is set to #e0e0e0: image

3. I can't seem to find anything that uses the setting "Action icon color" by the way, what is it used for?

TBog commented 3 years ago

I can't seem to find anything that uses the setting "Action icon color" by the way, what is it used for?

It's used for the contact call and message icon

TBog commented 3 years ago

It was (more) like the popup before, and I'd say the current implementation is quite a step backwards in terms of consistency/unity within the app.

The Settings Activity can't be customized, it just has a couple of themes. Why customize the Change icon dialog? Or the Edit QuickList dialog?

The QuickList dialog can be opened from the Settings and from the long-press pop-up in the QuickList. Initially it was only accessible from the settings and inherited the theme.

Help me create a Settings theme that you'd want to see for the custom icon dialog and QuickList editor.

ltguillaume commented 3 years ago

Simply because it's presented as a popup, not as a Settings screen: they are smaller dialogs on top of the activity you're in. This in contract to the Settings, which is a "full-screen" different Activity. That's why it's confusing that the (let's say "context menu") popups and these popups are so different.

ltguillaume commented 3 years ago

As for a Settings theme: these are the values I have for my popups, not sure if they translate nicely to a full-screen settings activity.

image

Corner radius: 0 Border: FF9E9E9E Background: CC212121 Touch: 424242 Text: E0E0E0 Title: EEEEEE

TBog commented 2 years ago

Now that I added the default theme, can we close this?

ltguillaume commented 2 years ago

No! 😛

  1. Edit tags still shows a green line around existing tags
  2. Edit tags still has an old [+] button
ltguillaume commented 2 years ago

No! 😛

1. Edit tags still shows a green line around existing tags

2. Edit tags still has an old green [+] button

(1) has been fixed, (2) is still present (but low-priority)

  1. The Add shortcut dialog is still completely green.
TBog commented 2 years ago

(1) has been fixed, (2) is still present (but low-priority)

  1. The Add shortcut dialog is still completely green.

The green [+] button is unique. I don't have a button elsewhere to copy the style from. I'm not sure how it should look, it's not my specialty.

I forgot about the confirm shortcut dialog. It's an unfinished dialog anyway. I wanted to be able to choose the icon and edit the label of the shortcut but only the label got implemented.

ltguillaume commented 6 months ago

Will the Add shortcut dialog ever be fixed? It's still toxic green.