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

Emergency Stop button position in the top bar #18

Closed
ARU64 opened this issue Dec 22, 2020 · 7 comments
Closed

Emergency Stop button position in the top bar #18

ARU64 opened this issue Dec 22, 2020 · 7 comments

Comments

@ARU64
Copy link

ARU64 commented Dec 22, 2020

With Octoprint 1.5.2, Python 3.7.3 and Octopi 0.18 the big estop or the small estop appear to low in the top bar.
image

@ZeekWorld
Copy link

Same with Dashboard plugin and UI Customizer
image

@Sebclem
Copy link
Owner

Sebclem commented Jan 4, 2021

Ok i can confirm that a conflict with UI Customizer, I will take a look at this and maybee open an issue on UI Customizer repo

@ZeekWorld
Copy link

Thank you.

@LazeMSS
Copy link

LazeMSS commented Jan 4, 2021

I think in general normally items to the top menu item is placed inside a single div to contain the element.

But im guessing the real problem is including a modal inside the header. If you want a confirm dialog then it should be placed at the end of the body/html.

A couple of other notes:
https://github.com/Sebclem/OctoPrint-SimpleEmergencyStop/tree/master/octoprint_simpleemergencystop/templates - you have "fontawesome.min.css" in her - this is wrong. its also includede here https://github.com/Sebclem/OctoPrint-SimpleEmergencyStop/tree/master/octoprint_simpleemergencystop/static/css which is the right place

Having inline style is bad practive when you have a css file:
https://github.com/Sebclem/OctoPrint-SimpleEmergencyStop/blob/master/octoprint_simpleemergencystop/templates/simpleemergencystop_navbar.jinja2#L2

@LazeMSS
Copy link

LazeMSS commented Jan 4, 2021

But i found something that I can try that might actually be better for both our plugins :)

@Sebclem
Copy link
Owner

Sebclem commented Apr 26, 2021

The remaining issue should be fixed with the last release.

Sorry for late fix, i was quite busy lately

@Sebclem
Copy link
Owner

Sebclem commented Apr 29, 2021

I'm closing this issue, feel free to re-open this issue or open a new one if you still have compatibility issue with UI Customizer

@Sebclem Sebclem closed this as completed Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants