Conversation
* autodetection of dark mode. The Linux solution was provided by ChatGPT and needs to be reviewed before being deployed. * default to system preset instead of light theme. * dark theme is mostly the ZWidget default, except that the garish orange (VkDoom leftover?) selection bar in list boxes has been replaced with a dark gray.
remove all AI-generated code, re-enable tooltips
|
I said about the tooltips that they still need work. When I ran the engine this afternoon the menu looked broken with one third of the screen at the bottom being empty and no idea why. You cannot ship a feature like that. People will flood your support with reports of things not working right. I only noticed what this was supposed to be when I finally opened a menu which had tooltips. Also, not being able to disable them entirely is quite an odd design decision. |
In the future people will be putting important information in them as we move away from awkward descriptors and huge blocks of static text that take up half the menu. Disabling them will just make things a confusing mess since now people will be relying on them for more contextual info and you'll find yourself having to enable them anyway. If you want to make the text small by default that's one thing, but I cannot see the vision of both disabling it by default and hiding the options menu entirely that allows tweaking it. This is basically just an attempt to hide the feature outright. Considering it was just added less than a week ago, I think it's understandable nothing was written for it yet when I wasn't even sure what the fate of the feature would be. Perhaps when the PR was left open for nearly a month you should've hopped in then instead of deciding to nuke it after the fact |
|
Nothing has been nuked. All I added was an option to switch the tooltips off. I understand your concerns - but in the end, if a user does not want this, they should have that option to disable it, especially since it reduces the available screen space. How about this: If a menu page has no tooltips, which can be statically checked up front, do not block the space. If it has some, leave the box on regardless of the existence of a tooltip for the current option. And since all the content is static so you can easily calculate up front how large a box you need and not oversize it if there is no content to fill it. If you know that you only need two lines why have a 3 line box? |
4 participants
remove all AI-generated code, re-enable tooltips