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

V8: Even out the padding in dialog footers #6800

Merged
merged 3 commits into from
Oct 29, 2019

Conversation

kjac
Copy link
Contributor

@kjac kjac commented Oct 21, 2019

Prerequisites

  • I have added steps to test this contribution in the description below

Description

Yeah so this is a super minor thing, but it's been bugging me so... here goes: The dialog footer padding is all in the bottom and nothing in the top. It's clearly visible when you have a scrollbar in the dialog.

Besides the lack of spacing above the footer buttons, these buttons are also misaligned with the content editor buttons:

image

As you might have guessed, this PR moves the dialog footer buttons a bit down and to the right to match the spacing of the content editor buttons. This also adds an appropriate amount of spacing above the dialog footer buttons:

image

@poornimanayar
Copy link
Contributor

@kjac ,
👍

@abjerner
Copy link
Contributor

@kjac thanks a lot 👍

This fixes the difference between the dialogs (left) and the overlays (right), so your PR is hereby approved.

Somewhat related, but probably something for another PR, there is however also some difference between the dialogs/overlays and the normal buttons in the bottom right corner:

image

@kjac
Copy link
Contributor Author

kjac commented Oct 24, 2019

@abjerner I'm not sure I follow... but there is also #6727 which centers around styling the overlays more V8-ish, does that solve it?

@abjerner
Copy link
Contributor

@kjac good question 😄

It don't think it does for this particular case. But I also noticed that the overlay I was checking had the old look and feel, so it probably shouldn't be used for testing.

Anyways - here is a bigger screenshot of what I'm referring to (there is a 2 pixel vertical difference):

image

Therefore I actually think the padding should be 8px instead of 10px - here's the result with 8px:

image

When changed to 8px, the buttons also align vertically with the image picker overlay as shown here:

image

What I was comparing against initially was the second overlay when inserting an image. The UI here more resembles the UI from Umbraco 7 overlays (which is a whole other issue - it may be what you're fixing in #6727):

image

So my conclusion is that I believe the padding should be 8px instead of 10px (although I approved the latter 🙃).

@nul800sebastiaan nul800sebastiaan merged commit 150cd07 into umbraco:v8/dev Oct 29, 2019
@nul800sebastiaan
Copy link
Member

Wonderful fixing of the details! 👍

@kjac kjac deleted the v8/fix/dialog-footer-padding branch October 29, 2019 20:57
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.

4 participants