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

Editor theme color renames break compatibility for plugins #88626

Closed
Jowan-Spooner opened this issue Feb 21, 2024 · 1 comment · Fixed by #89302
Closed

Editor theme color renames break compatibility for plugins #88626

Jowan-Spooner opened this issue Feb 21, 2024 · 1 comment · Fixed by #89302

Comments

@Jowan-Spooner
Copy link

Tested versions

4.3 dev3 (colors don't work), 4.3 dev2 (colors still work)

System information

non-relevant (Windows 10 though)

Issue description

#87085 has renamed many of the color theme items of the editor theme, others have been removed (Of course the difference between a removed and a renamed color are hard to tell, but some got obvious replacements).
It is common practice in editor plugins to make use of the colors/fonts/icons that the editor theme already provides. Any use of these (colors) will now break from 4.2 to 4.3. This is rather annyoing as having a plugin look correct on both of these versions becomes a very unncessary pain.

As one of the dialogic developers I noticed this change when testing the plugin with 4.3 dev 3. It is not present in 4.3 dev 2

Here are the colors in 4.2.x:
colors_old
And here in 4.3:
grafik

Notable renames are:

  • readonly_font_color -> font_readonly_color
  • disabled_font_color -> font_disabled_color
  • disabled_highlight_color -> highlight_disabled_color

Seemingly removed ones:

  • highlighted_font_color
  • property_color
  • readonly_color

There is a small chance this has never been intended to be used, however I find that unlikely as the person who made these changes had previously made a plugin to more easily see and copy these colors for use in plugins. Also they are very clearly accessibly with easy-to-use methods.

I have not checked whether there are also icon, font or stylebox renames/changes. that might break and have only checked the "Editor" section, not specific classes. However it is most likely that the items in the "Editor" category have been used by plugins in my experience.

Steps to reproduce

Use any of these colors in a project in 4.2 and 4.3 dev 3.

Minimal reproduction project (MRP)

N/A

@Mickeon
Copy link
Contributor

Mickeon commented Mar 8, 2024

Notable renames are:

  • readonly_font_color -> font_readonly_color
  • disabled_font_color -> font_disabled_color
  • disabled_highlight_color -> highlight_disabled_color

Seems pretty straightforward and doable to introduce aliases for these (and note them as such in the code so they can be safely removed when reasonable).
Not sure how reasonable it is to handle the rest, though, as chances are their removal is justified in some way. Perhaps they were too similar to other colors, potentially redundant or underutilised.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants