Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Stop inline-header filename from resizing. #5680

Closed
wants to merge 6 commits into from

Conversation

larz0
Copy link
Member

@larz0 larz0 commented Oct 25, 2013

This is for #5641.

screen shot 2013-10-24 at 5 03 59 pm

@redmunds
Copy link
Contributor

The file name size is fixed, but it looks like the header height still adjusts with the text size, to the close button doesn't stay aligned with the file name. This screen shot is after hitting Ctrl-+ 30 times and then turning on Dev Tools > Elements panel. Maybe try adjusting the baseline on the close button?

inline-header-resize

@larz0
Copy link
Member Author

larz0 commented Nov 1, 2013

@redmunds should be fixed now.

screen shot 2013-11-01 at 10 35 59 am

@@ -665,6 +665,11 @@ a, img {
}

/* Styles for inline editors */

.inline-text-editor {
line-height: 0px !important;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that fixes it.

I don't !important is needed here. I removed it and it still seems to work. Can we remove it?

@larz0
Copy link
Member Author

larz0 commented Nov 1, 2013

@redmunds !important removed~

@redmunds
Copy link
Contributor

redmunds commented Nov 1, 2013

Squashed these commits and merged in #5818. Closing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants