8000 Golden layout improvements by kebowen730 · Pull Request #1419 · holoviz/panel · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@kebowen730
Copy link
Contributor

Screen Shot 2020-06-17 at 3 14 08 AM
Screen Shot 2020-06-17 at 3 06 59 AM
I've attached screenshots of what the Golden Layout template looks like currently for feedback on the styling.

Starting with Phillip's initial template, I removed the original sidebar element and converted it into a Golden Layout Component. From there, I added styling and ran into an issue where I couldn't find a way to stratify which element was used for the built in Golden Layout light and dark themes by whether DarkTheme or DefaultTheme were set, and when I attempted to use an import in the respective css files the styling came out wrong and drastically different. Since the styling needs with golden layout were fairly complex, the solution I decided on was somewhat of a brute force approach where I copied the the contents of the built in stylesheets and copied them into the corresponding css files.

My solution works, but does anyone have an idea of a smoother way to handle that problem?

@philippjfr
Copy link
Member

I'm not really following your description here, but looking at the changes everything looks correct and as I would expect. Could you clarify what you mean by "stratify which element was used for the built in Golden Layout light and dark themes by whether DarkTheme or DefaultTheme were set , and when I attempted to use an import in the respective css files the styling came out wrong and drastically different."?

@kebowen730
Copy link
Contributor Author
kebowen730 commented Jun 17, 2020

@philippjfr I actually got the css imports working, I'm not sure what the issue was before but I'll have a new commit reflecting it soon.

@philippjfr
Copy link
Member

Can you explain how this PR relates to #1395? Does it supersede it? Should the other one be closed?

8000
@kebowen730
Copy link
Contributor Author

@philippjfr This does supersede #1395, I'll go close that PR.

@kebowen730
Copy link
Contributor Author

Screen Shot 2020-06-25 at 6 04 37 AM
Screen Shot 2020-06-25 at 6 03 27 AM

@philippjfr I made some updates to the html and styling to fix some border issues and restrict the width of the sidebar, as well as making the panel items fit within the tabs. Let me know if this looks correct to you.

@codecov
Copy link
codecov bot commented Jun 25, 2020

Codecov Report

Merging #1419 into master will decrease coverage by 0.02%.
The diff coverage is 75.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1419      +/-   ##
==========================================
- Coverage   87.78%   87.75%   -0.03%     
==========================================
  Files         135      136       +1     
  Lines       14930    14961      +31     
==========================================
+ Hits        13106    13129      +23     
- Misses       1824     1832       +8     
Impacted Files Coverage Δ
panel/template/bootstrap/__init__.py 100.00% <ø> (ø)
panel/template/material/__init__.py 100.00% <ø> (ø)
panel/template/base.py 55.93% <53.84%> (-0.32%) ⬇️
panel/template/golden/__init__.py 88.88% <88.88%> (ø)
panel/template/__init__.py 100.00% <100.00%> (ø)
panel/pane/vtk/synchronizable_serializer.py 84.46% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2f8c79...9682dab. Read the comment docs.

@philippjfr
Copy link
Member

Thanks, I'm pretty swamped but will try to review tomorrow.

@MarcSkovMadsen
Copy link
Collaborator

This template will be amazing. 👍

@philippjfr
Copy link
Member

Thanks @kebowen730! After a bit more cleanup and improvements to the resizing behavior this is working pretty well.

@philippjfr philippjfr merged commit bb28a7b into master Jul 3, 2020
@philippjfr philippjfr deleted the golden-layout-improvements branch July 3, 2020 13:56
philippjfr added a commit that referenced this pull request Sep 17, 2020
* attempting to nmove the sidebar content into a movable tab

* updating the sidebar to be a golden layout component

* solving an issue with golden layout by copying the golden layout light and dark themes into corresponding css themes to get around issues with importing the css files and using links in html

* fixing golden layout style sheet imports

* adding GoldenTemplate to the template __init__.py file

* removing unneeded imports

* updating golden template styling to restrict sidebar sizing, fix border issues and smooth out panel item widths

* Cleanup and layout improvements

* Improved resizing behavior

* Fixed flakes

* Label Golden Layout tabs

Co-authored-by: Philipp Rudiger <prudiger@anaconda.com>
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 12, 2025
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.

4 participants

0