search

rolandtoth / AdminOnSteroids

Various ProcessWire admin tweaks to boost productivity.
https://processwire.com/talk/topic/13389-adminonsteroids/
GNU General Public License v2.0
39 stars 10 forks source link

AOS with AdminThemeUikit prevents new page from Publishing #61

Closed gmclelland closed 7 years ago

gmclelland commented 7 years ago

This only happens when a brand new page is created and needs to be published.

Here is a screenshot of the top "Publish" button edit page test pwoliver dev 2017-10-02 12-04-03

Note: the bottom "Publish" button works just fine. edit page test pwoliver dev 2017-10-02 12-05-49

gmclelland commented 7 years ago

Ok, I found the problem.

If you disable the rules in the red box, everything starts to work correctly.

html.aos h1:not(.pwapi-headline) {
    position: relative;
}
html.aos h1 {
    max-width: calc(100% - 120px);
}

edit page hello pwoliver dev 2017-10-05 15-37-59

I'm not sure what those rules are for, but it doesn't appear to cause any problems with the Classic, Reno, or AdminThemeUikit when I disable them.

Hope that helps

gmclelland commented 7 years ago

I'll work on the rest tomorrow.

rolandtoth commented 7 years ago

The calc is there because when H1 is too long then it's cropped and an ellipsis is appended. On AdminthemeUikit the two buttons are much wider, that's why the overlap.

Thanks for the other PRs!

gmclelland commented 7 years ago

Ah, makes sense. Do you want to adjust the width or remove this functionality?

rolandtoth commented 7 years ago

I added max-width: calc(100% - 400px); above 960px screen width.

gmclelland commented 7 years ago

This is fixed now.