[ClayCSS] Table title horizontal alignment behavior differs from Chrome to Safari #2219
Labels
2.x
3.x
browser: safari
Issues that happen specifically in Safari browser
comp: clay-css
Issues related to Clay CSS
type: bug
Issues reporting that Component is not doing what should be done
Comments
diegonvs
added
browser: safari
|
Thanks for creating this @diegonvs! Fix should be merged to DXP as it affects all tables there. |
|
This is caused by #1743 |
pat270
added a commit
to pat270/clay
that referenced
this issue
Aug 21, 2019
… Safari Caused by liferay#1743 and fixes liferay#2219
pat270
added a commit
to pat270/clay
that referenced
this issue
Aug 21, 2019
… Safari Caused by liferay#1743 and fixes liferay#2219
This was referenced Aug 21, 2019
|
Create https://issues.liferay.com/browse/LPS-100174 for this issue. |
pat270
added a commit
to pat270/clay
that referenced
this issue
May 21, 2020
…ight` on the `html` element already set on `body` fix(@clayui/css): Reboot don't set `text-align: left` on `body` due to RTL issues. See liferay#1743 fix(@clayui/css): Reboot remove `:focus-visible` future-proof rule fix(@clayui/css): Reboot remove uses of Bootstrap's `font-size` and `hover` mixin, we don't support it in Clay CSS fix(@clayui/css): Reboot remove Bootstrap's `text-align` reset on `th`. See liferay#2219 fix(@clayui/css): Reboot remove Bootstrap's "hand" cursor to non-disabled button elements. We don't support `$enable-pointer-cursor-for-buttons`. We can change the cursor to whatever we want in Clay CSS. fix(@clayui/css): Reboot use selector `[href]` for styling anchor links instead of `a` so we don't need to overwrite placeholder anchor styles with `a:not([href])`. Styling `<a tabindex="0">a link</a>` should be done with a class like `link-primary`. Trying to avoid going down twbs/bootstrap#24140
pat270
added a commit
to pat270/clay
that referenced
this issue
Jun 29, 2020
…ight` on the `html` element already set on `body` fix(@clayui/css): Reboot don't set `text-align: left` on `body` due to RTL issues. See liferay#1743 fix(@clayui/css): Reboot remove `:focus-visible` future-proof rule fix(@clayui/css): Reboot remove uses of Bootstrap's `font-size` and `hover` mixin, we don't support it in Clay CSS fix(@clayui/css): Reboot remove Bootstrap's `text-align` reset on `th`. See liferay#2219 fix(@clayui/css): Reboot remove Bootstrap's "hand" cursor to non-disabled button elements. We don't support `$enable-pointer-cursor-for-buttons`. We can change the cursor to whatever we want in Clay CSS. fix(@clayui/css): Reboot use selector `[href]` for styling anchor links instead of `a` so we don't need to overwrite placeholder anchor styles with `a:not([href])`. Styling `<a tabindex="0">a link</a>` should be done with a class like `link-primary`. Trying to avoid going down twbs/bootstrap#24140
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The left page is running on Chrome and right, on Safari.
Same comparison on https://next.clayui.com/docs/components/table.html
Debugging, we are computing
text-align: start;on Safari andtext-align: inherit;on Chrome. Disabling this property on Safari's debugger, apparently fixed it.https://recordit.co/MxOMjKswBy
/cc @pat270 @brunofarache
The text was updated successfully, but these errors were encountered: