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

Breathe 2024 theme RTL support #377

Open
1 of 2 tasks
adamwoodnz opened this issue Sep 26, 2024 · 13 comments
Open
1 of 2 tasks

Breathe 2024 theme RTL support #377

adamwoodnz opened this issue Sep 26, 2024 · 13 comments
Assignees

Comments

@adamwoodnz
Copy link

adamwoodnz commented Sep 26, 2024

Audit the new Make blog theme (wporg-breathe-2024) for RTL support, in particular the /team sites, eg. https://ar.wordpress.org/team/

  • Log issues as comments below
  • Fix issues
@adamwoodnz adamwoodnz converted this from a draft issue Sep 26, 2024
@adamwoodnz adamwoodnz changed the title Breathe 2024 RTL support Breathe 2024 theme RTL support Sep 26, 2024
bazza pushed a commit that referenced this issue Oct 16, 2024
Change 404 page background color.

Fix some RTL issues.

See #377


git-svn-id: https://meta.svn.wordpress.org/sites/trunk@14108 74240141-8908-4e6f-9713-ba540dce6ec7
@adamwoodnz adamwoodnz moved this from 📋 To do to 🏗 In progress in WordPress.org Dec 2, 2024
@adamwoodnz adamwoodnz self-assigned this Dec 2, 2024
@adamwoodnz
Copy link
Author

Image

@adamwoodnz
Copy link
Author

Image

@adamwoodnz
Copy link
Author

Image

@adamwoodnz
Copy link
Author

Image

bazza pushed a commit that referenced this issue Dec 3, 2024
bazza pushed a commit that referenced this issue Dec 4, 2024
bazza pushed a commit that referenced this issue Dec 4, 2024
@adamwoodnz
Copy link
Author

Above issues are all fixed, leaving open in case anymore are found.

bazza pushed a commit that referenced this issue Dec 5, 2024
Fix issue with styling of o2 post form tabs.
Fix RTL display of mobile secondary fly out menu.
Fix RTL display of welcome box button.

Fixes #412
See #377


git-svn-id: https://meta.svn.wordpress.org/sites/trunk@14238 74240141-8908-4e6f-9713-ba540dce6ec7
@adamwoodnz
Copy link
Author

@ironnysh would you have any time for some RTL testing for us please? You can just leave comments below for anything you find 🙏

@adamwoodnz adamwoodnz moved this from 🏗 In progress to 📋 To do in WordPress.org Dec 8, 2024
@ironnysh
Copy link

ironnysh commented Dec 9, 2024

Hey @adamwoodnz, sure, happy to help. I'll review it tomorrow

@ironnysh
Copy link

Hey @adamwoodnz, here are my findings:

  1. Dropdown nav item is misaligned
    Image

  1. Switch the direction of the content and sidebar areas. By the way, why is the theme still using floats when we've been blessed by widely-supported flex and grid? :-)
    Image

  1. Bullets instead of avatars and extra padding on list items (sidebar)
    Image

  1. Extra bullets and wrong unicode arrow in the bottom nav (next-previous posts)
    Image

  1. Extra margin on the right in the footer
    Image

  1. Missing spacing on the team name in non-desktop breakpoints (889 pixels down to 320 pixels)
    Image

  1. Not sure if this is a theme or content issue, but the date-time format is wrong
    Image

@adamwoodnz
Copy link
Author

Thanks @ironnysh!

@adamwoodnz adamwoodnz moved this from 📋 To do to 🏗 In progress in WordPress.org Dec 10, 2024
@adamwoodnz adamwoodnz moved this from 🏗 In progress to 📋 To do in WordPress.org Dec 10, 2024
@adamwoodnz adamwoodnz moved this from 📋 To do to 🏗 In progress in WordPress.org Dec 10, 2024
@adamwoodnz adamwoodnz moved this from 🏗 In progress to 📋 To do in WordPress.org Dec 10, 2024
@adamwoodnz
Copy link
Author

2. Switch the direction of the content and sidebar areas. By the way, why is the theme still using floats when we've been blessed by widely-supported flex and grid? :-)

On LTR sites the sidebar is on the right. Should it be that way for RTL too? I expected it to swap.

This is an old theme that we are restyling, so there is still a lot of float magic around 🙂

@ironnysh
Copy link

On LTR sites the sidebar is on the right. Should it be that way for RTL too? I expected it to swap.

Exactly. Right now, the sidebar remains on the right also in RTL:
Image

The screenshot I pasted above is the desired end result, with the implemented CSS changes. Should have probably made it clearer :-)

Image

This is an old theme that we are restyling, so there is still a lot of float magic around 🙂

Fun with vintage! 😅 So, assuming it won't be updated to flex/grid, these are the styles that need to be updated:

.widget-area {
-float: right;
+float: left;
}
#secondary {
-clear: right;
+clear: left;
}
.content-area {
-float: left;
+float: right;
-padding-right: 20px;
+padding-left: 20px;

@adamwoodnz adamwoodnz moved this from 📋 To do to 🏗 In progress in WordPress.org Dec 11, 2024
@adamwoodnz
Copy link
Author

3. Bullets instead of avatars and extra padding on list items (sidebar)

Those are actually in the content

Image

@adamwoodnz
Copy link
Author

4. Extra bullets and wrong unicode arrow in the bottom nav (next-previous posts)

Same with these bullets, the post titles appear to include them

Image

bazza pushed a commit that referenced this issue Dec 11, 2024
Primary and secondary column order.
List padding.
Local nav padding.

See #377


git-svn-id: https://meta.svn.wordpress.org/sites/trunk@14268 74240141-8908-4e6f-9713-ba540dce6ec7
@adamwoodnz adamwoodnz moved this from 🏗 In progress to 📋 To do in WordPress.org Dec 11, 2024
bazza pushed a commit that referenced this issue Dec 11, 2024
New RTL styles didn't account for this layout using the same selectors.

See #377


git-svn-id: https://meta.svn.wordpress.org/sites/trunk@14269 74240141-8908-4e6f-9713-ba540dce6ec7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📋 To do
Development

No branches or pull requests

2 participants