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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix the back end layout yet again #1879

Merged
merged 4 commits into from Jun 29, 2020

Conversation

fritzmg
Copy link
Contributor

@fritzmg fritzmg commented Jun 26, 2020

This is a follow up to #1823, #1875 and #1877

Some previous changes have been reverted, some new changes have been added.

  • Text inputs and selects and submit buttons now have the same height (and they are aligned correctly).
  • Layout does not break for certain w50 widget combinations in any browser, zoom level and dppx combination that I have tested (tested with newest Firefox, Chrome and Edge).

@leofeyer can you test these changes in Safari too please? Hopefully this is now fixed for good and we don't have to touch it again, until we re-do the whole thing with CSS grids 馃榿

If everything checks out, I'll update the PR for Contao 4.4 accordingly.

@fritzmg fritzmg added the bug label Jun 26, 2020
@fritzmg fritzmg added this to the 4.9 milestone Jun 26, 2020
@fritzmg fritzmg requested a review from leofeyer June 26, 2020 13:38
@fritzmg fritzmg self-assigned this Jun 26, 2020
@leofeyer
Copy link
Member

Also, we can remove the search field padding workaround for Firefox again: https://github.com/contao/contao/pull/1875/files#diff-6d58d2e9400bef23b3f9ae54a41f666bR306-R309

@fritzmg fritzmg requested a review from leofeyer June 29, 2020 13:36
@leofeyer leofeyer merged commit 9e8500f into contao:4.9 Jun 29, 2020
@leofeyer
Copy link
Member

Thank you @fritzmg.

@leofeyer leofeyer changed the title Change back end layout yet again Fix the back end layout yet again Jul 9, 2020
@fritzmg fritzmg deleted the fix/revert-previous-changes branch October 24, 2021 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants