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

[WICKET-7068] Tree styles are updated to be RTL-friendly #600

Merged
merged 1 commit into from
Aug 23, 2023

Conversation

solomax
Copy link
Contributor

@solomax solomax commented Aug 19, 2023

This one can be tested at https://examples9x.wicket.apache.org/tree by setting dir="rtl" attribute for <html> tag

I'll create JIRA if this PR will be considered useful :)

@reiern70
Copy link
Contributor

@solomax Maybe create a second example page with dir="rtl"

@solomax solomax requested a review from martin-g August 21, 2023 06:57
@solomax
Copy link
Contributor Author

solomax commented Aug 21, 2023

I'm not sure what would be the best way in this case :(

It might be some sort of dropdown (so every page can be checked in RTL), but this might be useless due to all contents is in English ....

Copy link
Member

@martin-g martin-g left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks OK to me but I am not the best person to review CSS code!

@solomax solomax changed the title Tree styles are updated to be RTL-friendly [WICKET-7068] Tree styles are updated to be RTL-friendly Aug 21, 2023
@solomax solomax requested a review from bitstorm August 22, 2023 14:17
@solomax
Copy link
Contributor Author

solomax commented Aug 23, 2023

I'm going to merge this one :)

@solomax solomax merged commit 45b91d4 into wicket-9.x Aug 23, 2023
2 checks passed
@solomax solomax deleted the tree-rtl-styling branch August 23, 2023 15:46
@solomax solomax restored the tree-rtl-styling branch August 24, 2023 03:39
@martin-g
Copy link
Member

@solomax Should this be ported to master ?

@solomax
Copy link
Contributor Author

solomax commented Aug 24, 2023

@solomax Should this be ported to master ?

Unfortunately while cherry-picking and testing I found minified CSS is broken :(

please see #602

This one should be additionally fixed by merging #602 or reverted .... :((

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

Successfully merging this pull request may close these issues.

3 participants