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

Theme that replicates the coloring of the WTP XML editor #572

Closed
kevloral opened this issue Jul 28, 2023 · 10 comments
Closed

Theme that replicates the coloring of the WTP XML editor #572

kevloral opened this issue Jul 28, 2023 · 10 comments

Comments

@kevloral
Copy link
Contributor

Having a theme that replicated the coloring scheme used in the WTP XML editor would be nice.

@angelozerr
Copy link
Contributor

Any contribution are welcome!

@kevloral
Copy link
Contributor Author

The theme is called "Traditional" and it is quite close to the syntax coloring scheme used in the WTP XML editor. It is not perfect, though, as some elements that would colored differently in the original editor share the same CSS here. For the time being, the only difference I have been able to find can be seen on this screenshot, but there could be more:

image

In the original WTP editor, the equals sign would be black, but here that sign shares the same CSS (.meta.tag) as the tag delimiters.

@angelozerr
Copy link
Contributor

The theme is called "Traditional" and it is quite close to the syntax coloring scheme used in the WTP XML editor.

You say that it is Traditional because you think about WTP. But this theme is not "Traditional" for Java code color. It is Tradionnal for WTP but not for other editor. It is the reason why I suggested to rename to WTP.

@sebthom what do you think about that?

It is not perfect

It is a good start! Thanks so much for your contribution.

@sebthom
Copy link
Member

sebthom commented Jul 28, 2023

If the theme is specific for XML files, maybe it should be added to the language pack instead of the default UI plugin and be named "WTP XML Classic" or something like this?

@kevloral
Copy link
Contributor Author

You are both right. I named it Traditional because, as you may imagine, I have started using this XML editor after having used the WTP XML editor for years. Therefore, I agree something along the lines of "WTP XML Classic" would be much more appropriate.

@angelozerr
Copy link
Contributor

If the theme is specific for XML files,

It is specific for WTP, in other words CSS, XML, HTML.

Therefore, I agree something along the lines of "WTP XML Classic" would be much more appropriate.

+1

maybe it should be added to the language pack

I have not strong opinion about that

@sebthom
Copy link
Member

sebthom commented Jul 28, 2023

We could also just call the theme WTP or Eclipse WTP if it was used by WTP for more than just XML files.

@kevloral
Copy link
Contributor Author

Please note that I have only tested the new theme with XML files.

@sebthom
Copy link
Member

sebthom commented Aug 14, 2023

@kevloral you seemed to have closed your PR. Is this enhancement not needed anymore?

@kevloral
Copy link
Contributor Author

My bad. I thought it had already been merged. I have a local copy of the .css file, so I will use it to create a new PR for this.

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

No branches or pull requests

3 participants