-
Notifications
You must be signed in to change notification settings - Fork 60
Filament fields page cutoff (UI Customizer plugin) #142
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
Comments
which browser are you using? maybe some adblocker cuts it off, I'm on the same versions and everything seems fine (see reply on #123) - try incognito mode (browser plugins disabled per default) or other browsers. I'm on google-chrome 89.0.4389.114 for reference, OctoPi version: 0.18.0, OctoPrint: 1.5.3, SpoolManager Plugin: 1.3.3; both before and after an |
I have uBlock Origin but disabled. I do have Themeify installed. After my current print is done, I'll disable that plugin and try again. |
is it maybe caused by zoom level and UI customizer as in #112 ? |
Because I can't reproduce the issue I need more infos:
|
This problem occurs on my PC with both Firefox (latest) and Edge (latest). Resolution: 1920x1200 Browser Zoom: 110%. If I reduce the zoom level down in steps to 50%, the missing elements start to progressively reveal themselves. If I increase the zoom level to 200% in steps, again, the missing elements progressively reveal themselves. OS: Windows 10 Pro build 19042.928 on desktop PC Plug-ins: | Action Command Notification Support (bundled) = /home/pi/oprint/lib/python3.7/site-packages/octoprint/plugins/action_command_notification On my iPad (2020 iPad Air 4), the Spools page loads OK in both Firefox and Safari. Hope this helps. |
I have the exact same issue on Brave |
@OllisGit its caused by this: OctoPrint-SpoolManager/octoprint_SpoolManager/templates/SpoolManager_tab_dialogs.jinja2 Line 167 in 0e86e1b
That hidden input breaks the before stuff of the way the responsive bootstrap works. If you move that to right after the form ( OctoPrint-SpoolManager/octoprint_SpoolManager/templates/SpoolManager_tab_dialogs.jinja2 Line 164 in 0e86e1b
I would also suggest you change the OctoPrint-SpoolManager/octoprint_SpoolManager/templates/SpoolManager_tab_dialogs.jinja2 Line 165 in 0e86e1b
Ohh and maybe expand the editor: OctoPrint-SpoolManager/octoprint_SpoolManager/templates/SpoolManager_tab_dialogs.jinja2 Line 418 in 0e86e1b
to span12 |
Hi, Please test and give me a feedback. Thx, in advance |
The missing fields problem is back with 1.4.0. All other plugins are the latest versions of my list from above. Zooming out to 67% or in to 200% gradually reveals the missing fields in a single column. |
Just tried with 1.4.1 (and now 1.4.2) as well and the issue persists. I have restarted the Raspberry Pi and Octoprint. Updated Plugin List | Action Command Notification Support (bundled) = /home/pi/oprint/lib/python3.7/site-packages/octoprint/plugins/action_command_notification My screenshots are the same as the first post above. I disabled all plug-ins except UI-Customiser and the issue persists. Disabling Ui-Customiser clears the issue. HTH. |
Agreed - something is more broken now |
Not sure why but my right colum has span12 and not span6 |
it must be something in my code I think... Update: Yes its the old workaround I made that now gives conflicts. |
@OllisGit I have just released a new version - was a bit rushed but didn't want your plugin to get bad rep due to my "hacks" |
Hi @LazeMSS, I installed the latest version 0.1.5.9 (wow, 4 digits) and can confirm that the spool-item dialog is "almost" fixed (the active checkbox is not in the upper right corner). But I also tested the PrintJob-Dialog (V1.14.0) and this dialog is still broken. Everything is in one column. |
Changing the right column from |
I hopefully have a couple of hours this weekend. I will try and do a rework of the UI and send you a pr |
Hi @LazeMSS, I played around with the chrome dev-tools and saw that your plugin overwrite the standard bootstrap 'span' classes (puhh, is that the right strategy). For the PJH-Dialog I disabled the span6 class and now I can see the two columns again. As I mentioned I am not a UI-Export, but I think this is not a good way to do. Of course you did an AMAZING job to do a total redesign of the default UI, but are there some other ways to do? Maybe this is also a question to the OP-Community how to separate Plugin UI-Stuff, is "Shadow Dow" the way to go I appreciate it very much that you try to find free time to analyse the issue a little bit deeper and (probably) create a PR! BR |
My plugin basically just include the standard bootstrap responsive files. It's a standard way of doing it in bootstrap. Your dialog is basically to crowded for at bootstrap modal dialog. I will do my best to make a clean solution for you ASAP. |
fixed at my Safari browser on MacBookPro |
@OllisGit don't know what PrintJob-Dialog is? |
It also works fine for me :( |
fyi: PJH == PrintJobHistory-Plugin --> https://github.com/OllisGit/OctoPrint-PrintJobHistory |
- PR #204, #201 redesign of the Spool-Selection in the sidebar. Huge thanks to @TiziG @MarloSM - E #125 New field "Serialnumber" - E Spool selection from Edit-Dialog - E #141 New Filter: Material, Vendor and Color - E #44 Sidebar filament usage in grams instead of volume - B #199, #198, #163, #156 some QR-Code optimisations - B #197, #196, #188, #181, #178, #177, #172, #162, #142, Some UI optimisations - B #194 CSV import, behind reverse proxy - B #175 Confirm dialog not shown
Hi all , Please try and give me a feedback. |
No feedback, please raise new issue if needed |
I am unable to access some of the fields on the bottom of the screen. This is the only plugin with this behavior and is not effected by an update, different (new and old) filaments. It could be specific to my Octoprint setup but I believe I have eliminated other variables.

OctoPrint: 1.5.3
SpoolManager Plugin: 1.3.3
The text was updated successfully, but these errors were encountered: