TBLauncher icon indicating copy to clipboard operation
TBLauncher copied to clipboard

Change the launcher's (too) bright green colors

Open ltguillaume opened this issue 4 years ago • 42 comments

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

ltguillaume avatar Dec 28 '20 19:12 ltguillaume

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?

TBog avatar Dec 28 '20 21:12 TBog

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.

ltguillaume avatar Dec 28 '20 22:12 ltguillaume

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?

TBog avatar Dec 28 '20 22:12 TBog

  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.

ltguillaume avatar Dec 28 '20 22:12 ltguillaume

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.

TBog avatar Dec 28 '20 22:12 TBog

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().

ltguillaume avatar Dec 28 '20 23:12 ltguillaume

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 avatar Dec 28 '20 23:12 TBog

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.

TBog avatar Dec 28 '20 23:12 TBog

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.

ltguillaume avatar Dec 28 '20 23:12 ltguillaume

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 avatar Jan 14 '21 11:01 TBog

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

TBog avatar Jan 14 '21 18:01 TBog

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

ltguillaume avatar Jan 14 '21 19:01 ltguillaume

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.

ltguillaume avatar Jan 14 '21 21:01 ltguillaume

  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.

TBog avatar Jan 14 '21 21:01 TBog

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

  • Primary text/titles: #e0e0e0
  • Secondary text/scroll bar: #9e9e9e

Not sure if it looks great right away.

ltguillaume avatar Jan 14 '21 21:01 ltguillaume

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

TBog avatar Jan 18 '21 17:01 TBog

(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).

ltguillaume avatar Jan 18 '21 17:01 ltguillaume

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?

TBog avatar Jan 18 '21 18:01 TBog

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.

ltguillaume avatar Jan 18 '21 18:01 ltguillaume

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?

TBog avatar Jan 18 '21 18:01 TBog

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").

ltguillaume avatar Jan 18 '21 19:01 ltguillaume

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

TBog avatar Jan 18 '21 19:01 TBog

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.

ltguillaume avatar Jan 18 '21 19:01 ltguillaume

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

TBog avatar Jan 21 '21 11:01 TBog

Fantastic! Thank you, great work!

ltguillaume avatar Jan 21 '21 12:01 ltguillaume

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

TBog avatar Jan 22 '21 20:01 TBog

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.

ltguillaume avatar Jan 23 '21 15:01 ltguillaume

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.

TBog avatar Jan 23 '21 20:01 TBog

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 avatar Jan 23 '21 21:01 ltguillaume

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 avatar Jan 29 '21 03:01 ltguillaume

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

  • 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.
  • 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: image

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).

ltguillaume avatar Feb 02 '21 23:02 ltguillaume

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.

TBog avatar Feb 02 '21 23:02 TBog

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.

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

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

ltguillaume avatar Feb 02 '21 23:02 ltguillaume

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 avatar Feb 03 '21 00:02 TBog

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.

TBog avatar Feb 03 '21 00:02 TBog

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 avatar Feb 03 '21 00:02 ltguillaume

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

ltguillaume avatar Feb 03 '21 00:02 ltguillaume

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

TBog avatar Mar 31 '22 18:03 TBog

No! 😛

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

ltguillaume avatar Apr 02 '22 12:04 ltguillaume

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.

ltguillaume avatar May 24 '22 17:05 ltguillaume

(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.

TBog avatar May 24 '22 18:05 TBog