Skip to content

Make sidebar collapsible #204

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

Merged
merged 1 commit into from
Oct 17, 2021
Merged

Make sidebar collapsible #204

merged 1 commit into from
Oct 17, 2021

Conversation

MarloSM
Copy link
Contributor

@MarloSM MarloSM commented Oct 11, 2021

After looking at #201, I realise that simply removing the overflow won't do. Addresses #197

Changing it to #sidebar_plugin_SpoolManager.collapse.in makes it work.

After looking at OllisGit#201, I realise that simply removing the overflow won't do. Changing it to #sidebar_plugin_SpoolManager.collapse.in makes it work.
@OllisGit OllisGit added the status: inNextRelease Will be implemented/fixed in next release label Oct 17, 2021
@OllisGit
Copy link
Owner

Hi @MarloSM,
thanks a lot for your effort!!!

I tested your PR a couple of days ago it worked for me and I was building a new release, but now I saw that @tiz also changed his/her PR https://github.com/OllisGit/OctoPrint-SpoolManager/pull/201/commits

Before I continue building the new release I will take a look on this PR.

@OllisGit OllisGit merged commit 00293e9 into OllisGit:master Oct 17, 2021
OllisGit added a commit that referenced this pull request Oct 24, 2021
- 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
@OllisGit OllisGit removed the status: inNextRelease Will be implemented/fixed in next release label Oct 24, 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

Successfully merging this pull request may close these issues.

2 participants