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

Fix th alignment for Safari #30323

Merged
merged 1 commit into from
Mar 2, 2020
Merged

Fix th alignment for Safari #30323

merged 1 commit into from
Mar 2, 2020

Conversation

MartijnCuppens
Copy link
Member

Fixes #30321.

Did some tests and it seemed only Safari had issues figuring out the default text alignment.

@MartijnCuppens MartijnCuppens requested a review from a team as a code owner March 1, 2020 18:06
@XhmikosR
Copy link
Member

XhmikosR commented Mar 1, 2020

Shouldn't this be backported?

@XhmikosR XhmikosR merged commit 4815408 into master Mar 2, 2020
@XhmikosR XhmikosR deleted the master-mc-th-alignment branch March 2, 2020 06:36
brianteeman added a commit to brianteeman/joomla-cms that referenced this pull request May 3, 2020
PR for joomla#28904

This is a test PR for the TH alignment issue with safari. It is a bug in bootstrap that they have fixed and merged but not released yet. See twbs/bootstrap#30323

This PR adds a _temporary_ reboot.css override with the code direct from bootstrap. I can't see any other way to fix this beta blocker.

I do not have access to Safari to test this.

@nikosdion please can you test this
infograf768 pushed a commit to joomla/joomla-cms that referenced this pull request May 4, 2020
* [4.0] th on safari

PR for #28904

This is a test PR for the TH alignment issue with safari. It is a bug in bootstrap that they have fixed and merged but not released yet. See twbs/bootstrap#30323

This PR adds a _temporary_ reboot.css override with the code direct from bootstrap. I can't see any other way to fix this beta blocker.

I do not have access to Safari to test this.

@nikosdion please can you test this

* Update _reboot.scss
olsza pushed a commit to olsza/bootstrap that referenced this pull request Oct 3, 2020
sakiss pushed a commit to sakiss/joomla-cms that referenced this pull request Oct 16, 2020
* [4.0] th on safari

PR for joomla#28904

This is a test PR for the TH alignment issue with safari. It is a bug in bootstrap that they have fixed and merged but not released yet. See twbs/bootstrap#30323

This PR adds a _temporary_ reboot.css override with the code direct from bootstrap. I can't see any other way to fix this beta blocker.

I do not have access to Safari to test this.

@nikosdion please can you test this

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

Successfully merging this pull request may close these issues.

Table headers and text-align
3 participants