-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Lib redesign skin fixes #1671
Merged
Be-ing
merged 32 commits into
mixxxdj:jmigual-library-redesign
from
ronso0:lib-redesign-skin-fixes
Jun 23, 2018
Merged
Lib redesign skin fixes #1671
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
1c31d7d
lib-redesign Deere: move Lib expand toggle above sidebar buttons
f08c2ca
lib-redesign LateNight: fix Library styling
4b978c3
lib-redesign LateNight: add CO for Lib icon text to skin.xml
b0627dd
lib-redesign Tango: fix & comment Library styling
c4a7b5c
lib-redesign LateNight: fix inactive library buttons
a284bf0
lib-redesign LateNight: add Id to LibrarySidebarButtons
0f820c3
lib-redesign LateNight: small UX tweaks
22e4fc0
LateNight/Tango: tweak library sort buttons
4dd9849
lib-redesign Deere: hide feature label
983e208
lib-redesign Deere: fix library feature top margin
2b5ec51
lib-redesign Deere: style AutoDJ transition time like beatsize spinboxes
4824f42
lib-redesign Deere: emphasize tab buttons in library feature
8b3d5f7
lib-redesign LateNight: properly hide the feature label
265b248
lib-redesign LateNight: style feature tab header buttons
4fa873e
lib-redesign LateNight: style buttons, remove double border in AutoDJ…
5690baf
lib-redesign Deere: revert Feature label hack for recording feature
3c264c2
lib-redesign Tango: make Analysis radio buttons smaller
f13eca8
lib-redesign Tango: style AutoDJ spinbox like beatsize spinboxes
a2b0477
lib-redesign Tango: clean up
c659ddb
lib-redesign Shade: remove label above feature pane, remove old QPush…
2b06492
lib-redesign: clean up code, rename lib sidebar width detector and sp…
d46c33c
lib-redesign Tango: move Breadcrumbs above searchbox like in other sk…
c711bd8
lib-redesign LateNight/Tango: fix some indentations, add comments
161956a
lib-redesign LateNight: brighter text for selected tree/table items
db189a4
lib-redesign skins: resolve merge conflicts
4bbbe0f
lib-redesign Shade: fix Library spacing & scrollbars, fix indentation…
680a701
lib-redesign skins: allow to toggle Library button text in all skins
682f0c8
lib-redesign skins: small fixes
86fc0f3
lib-redesign Shade: style Library features like rest of skin, incl. c…
330d364
merge jmigual-library-redesign, fix conflicts again
41115f2
lib-redesign skins: fix regressions introduced by git auto-merge
2b0a466
lib-redesign-skins: fix Splitter regressions
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
76 changes: 76 additions & 0 deletions
76
res/skins/LateNight/buttons/btn_lib_radio_button_off_hover.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,24 +1,24 @@ | ||
<Template> | ||
<WidgetGroup> | ||
<ObjectName>Library</ObjectName> | ||
<SizePolicy>me,i</SizePolicy> | ||
<MinimumSize>1270,-</MinimumSize> | ||
<Layout>vertical</Layout> | ||
<Children> | ||
|
||
<WidgetGroup><Size>0me,5f</Size></WidgetGroup> | ||
<!-- | ||
<WidgetGroup><Size>0me,5f</Size></WidgetGroup> | ||
********************************************** | ||
Library / Preview Deck / Search / Tree view | ||
********************************************** | ||
--> | ||
<WidgetGroup> | ||
<ObjectName>Library</ObjectName> | ||
<Layout>horizontal</Layout> | ||
<SizePolicy>me,me</SizePolicy> | ||
<Children> | ||
|
||
<Splitter> | ||
<ObjectName>LibrarySplitter</ObjectName> | ||
<ObjectName>VertSplitter</ObjectName> | ||
<SizePolicy>me,me</SizePolicy> | ||
<SplitSizes>1,10</SplitSizes> | ||
<SplitSizesConfigKey>[LateNight],LibrarySidebarSplitSize</SplitSizesConfigKey> | ||
|
@@ -32,45 +32,77 @@ | |
|
||
<Template src="skin:preview_deck.xml"/> | ||
|
||
<!--Search Input Field--> | ||
<SearchBox></SearchBox> | ||
|
||
<Splitter> | ||
<ObjectName>CoverArtSplitter</ObjectName> | ||
<Size>me,me</Size> | ||
<SplitSizes>1,1</SplitSizes> | ||
<SplitSizesConfigKey>[LateNight],CoverArtSplitSize</SplitSizesConfigKey> | ||
<Orientation>vertical</Orientation> | ||
<Collapsible>0,0</Collapsible> | ||
<WidgetGroup> | ||
<Layout>horizontal</Layout> | ||
<Children> | ||
<!--Library Tree View--> | ||
<LibrarySidebar></LibrarySidebar> | ||
<!--Cover Art--> | ||
<CoverArt> | ||
<SizePolicy>me,me</SizePolicy> | ||
<MinimumSize>40,40</MinimumSize> | ||
<Connection> | ||
<ConfigKey persist="true">[Library],show_coverart</ConfigKey> | ||
<BindProperty>visible</BindProperty> | ||
</Connection> | ||
</CoverArt> | ||
<WidgetGroup> | ||
<ObjectName>LibrarySidebarButtonsContainer</ObjectName> | ||
<Layout>vertical</Layout> | ||
<Children> | ||
<LibrarySidebarButtons> | ||
<SizePolicy>min,me</SizePolicy> | ||
</LibrarySidebarButtons> | ||
</Children> | ||
</WidgetGroup> | ||
|
||
<Splitter> | ||
<ObjectName>HoriSplitter</ObjectName> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Please spell out "HorizontalSplitter" |
||
<Size>me,me</Size> | ||
<SplitSizes>1,1</SplitSizes> | ||
<SplitSizesConfigKey>[LateNight],CoverArtSplitSize</SplitSizesConfigKey> | ||
<Orientation>vertical</Orientation> | ||
<Collapsible>0,0</Collapsible> | ||
<Children> | ||
<!--Library Tree View--> | ||
<LibrarySidebarExpanded> | ||
<ObjectName>LibrarySidebarExpanded</ObjectName> | ||
<Id>1</Id> | ||
</LibrarySidebarExpanded> | ||
<!--Cover Art--> | ||
<CoverArt> | ||
<SizePolicy>me,me</SizePolicy> | ||
<MinimumSize>40,40</MinimumSize> | ||
<Connection> | ||
<ConfigKey persist="true">[Library],show_coverart</ConfigKey> | ||
<BindProperty>visible</BindProperty> | ||
</Connection> | ||
</CoverArt> | ||
</Children> | ||
</Splitter> | ||
</Children> | ||
</Splitter> | ||
</WidgetGroup> | ||
|
||
</Children> | ||
</WidgetGroup> | ||
|
||
<!-- Library Table--> | ||
<Library> | ||
<BgColor>#585858</BgColor> | ||
<FgColor>#eece33</FgColor> | ||
</Library> | ||
<!-- Breadcrumbs + Searchbox + Library Table--> | ||
<WidgetGroup> | ||
<Layout>vertical</Layout> | ||
<Children> | ||
<LibraryBreadCrumb> | ||
<ObjectName>LibraryBreadCrumb</ObjectName> | ||
<SizePolicy>me,min</SizePolicy> | ||
<Id>1</Id> | ||
</LibraryBreadCrumb> | ||
|
||
<SearchBox> | ||
<Id>1</Id> | ||
</SearchBox> | ||
|
||
<LibraryPane> | ||
<BgColor>#585858</BgColor> | ||
<FgColor>#eece33</FgColor> | ||
<Id>1</Id> | ||
</LibraryPane> | ||
</Children> | ||
</WidgetGroup> | ||
|
||
</Children> | ||
</Splitter> | ||
|
||
</Children> | ||
</WidgetGroup> | ||
|
||
</Children> | ||
</WidgetGroup> | ||
</Template> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please spell out "VerticalSplitter"