-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
UI Update: WIP #2013
Merged
michael-hawker
merged 55 commits into
CommunityToolkit:master
from
shweaver-MSFT:ui-update
May 24, 2018
Merged
UI Update: WIP #2013
Changes from all commits
Commits
Show all changes
55 commits
Select commit
Hold shift + click to select a range
f230303
Fix crash (#1913).
azchohfi d360443
Fluentized BladeView control and fixed collapsed/expanded functionality
shweaver-MSFT 1657cdc
Fluentized Expander control
shweaver-MSFT 121b2b8
Fluentized BladeView control and fixed collapsed/expanded functionality
shweaver-MSFT 97d3770
Merge branch 'shweaver/expander' into shweaver/fluent
shweaver-MSFT cf345fc
Merge branch 'shweaver/bladeview' into shweaver/fluent
shweaver-MSFT 3862089
Updated GridSplitter control
shweaver-MSFT d60ac19
Sample page tweaks
shweaver-MSFT fae6b47
Merge branch 'shweaver/gridsplitter' into ui-update
shweaver-MSFT 3e7693b
Fluentize InAppNotifications control
shweaver-MSFT 451e041
Merge branch 'shweaver/inappnotifications' into ui-update
shweaver-MSFT 4a43744
Updated FileMenu control template
shweaver-MSFT ed4007f
Merge branch 'shweaver/filemenu' into ui-update
shweaver-MSFT 8a8b1ce
Menu sample tweaks
shweaver-MSFT b5e6d8f
Updated OrbitView design
shweaver-MSFT 91ce35b
Merge branch 'shweaver/orbitview' into ui-update
shweaver-MSFT 450f3b9
Migrating Dashoe's upated RangeSelector into the toolkit
shweaver-MSFT d6dc69b
Merge branch 'shweaver/rangeselector' into ui-update
shweaver-MSFT e7a5e27
Updated TextToolbar template
shweaver-MSFT a450a95
Merge branch 'shweaver/texttoolbar' into ui-update
shweaver-MSFT eba35b0
wip
shweaver-MSFT 7103a41
Added border to Menu and moved AcrylicBrush to code with type safety …
shweaver-MSFT ad6fe28
Added null checks for RangeSelector tooltip and value textBlocks to e…
shweaver-MSFT c4951ff
RadialGauge design update
shweaver-MSFT f3673e9
Merge branch 'shweaver/radialgauge' into ui-update
shweaver-MSFT f08f683
Fix Expander to have proper height when collapsed and no content overlay
shweaver-MSFT a1476e0
Color updates to Expander
shweaver-MSFT 057f118
Swapped out colors for ThemeResources in GridSplitter sample page
shweaver-MSFT 586f01f
Merge branch 'rel/3.0.0-preview' into ui-update
shweaver-MSFT 18a59c1
Fixed Menu commands
shweaver-MSFT 65afbab
Reset RadialGauge property defaults and removed unecessary content fr…
shweaver-MSFT c74e70f
Fixed buttons offset in InAppNotification template and fixed dismiss …
shweaver-MSFT f61789a
Merge branch 'rel/3.0.0-preview' into ui-update
shweaver-MSFT 6f59dd2
Merge branch 'onovotny/tpmv-update' into ui-update
shweaver-MSFT e0b4866
Fixed Acrylic in control templates using Conditional XAML
shweaver-MSFT babb82c
Merge branch 'rel/3.0.0-preview' into ui-update
nmetulev 70805b7
PR fixes
shweaver-MSFT 5296ef1
Added ShowValues property to RangeSelector for visibility toggling. D…
shweaver-MSFT ac515b9
Resetting RangeSelector default values
shweaver-MSFT af0fecc
BladeView Dark Theme updates
shweaver-MSFT af51ee8
Expander Dark Theme updates
shweaver-MSFT 3088c04
Menu Dark Theme updates
shweaver-MSFT 3732e13
InAppNotification Dark Theme updates
shweaver-MSFT 2b98d64
Merge branch 'master' into ui-update
shweaver-MSFT 6986707
RangeSelector Dark Theme updates. Added Background to tooltip
shweaver-MSFT 75b4466
RangeSelector color fixes
shweaver-MSFT 5e11a42
Merge branch 'master' into ui-update
shweaver-MSFT ce774ef
Color updates for RadialGauge control
shweaver-MSFT 2702a70
Merge branch 'master' into ui-update
nmetulev 66bd14f
RangeSelector tooltip bug fix
shweaver-MSFT 3d656d0
Merge branch 'ui-update' of https://github.com/shweaver-MSFT/UWPCommu…
shweaver-MSFT b4496e5
Merge branch 'master' into ui-update
shweaver-MSFT d1fc2c4
Merge branch 'master' into ui-update
nmetulev 8cc2380
Merge branch 'master' into ui-update
nmetulev f98e93c
Merge branch 'master' into ui-update
nmetulev 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
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
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
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 |
---|---|---|
|
@@ -6,46 +6,46 @@ | |
xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" | ||
mc:Ignorable="d"> | ||
|
||
<Page.Resources> | ||
<SolidColorBrush Color="{StaticResource SystemChromeLowColor}" x:Key="SystemControlForegroundChromeLowBrush"/> | ||
</Page.Resources> | ||
|
||
<Grid Background="{ThemeResource ApplicationPageBackgroundThemeBrush}"> | ||
<ScrollViewer> | ||
<StackPanel Margin="20"> | ||
<controls:Expander x:Name="Expander1" VerticalAlignment="Top" Margin="0,0,0,10" | ||
Header="This is the header - expander 1" | ||
HorizontalContentAlignment="Stretch" | ||
Foreground="White" | ||
Background="@[BackgroundExpander1:Brush:Gray]" | ||
IsExpanded="@[IsExpanded1:Bool:False]@" | ||
ExpandDirection="@[ExpandDirection1:Enum:ExpandDirection.Down]"> | ||
Header="This is the header - expander 1" | ||
HorizontalContentAlignment="Stretch" | ||
IsExpanded="@[IsExpanded1:Bool:False]@" | ||
ExpandDirection="@[ExpandDirection1:Enum:ExpandDirection.Down]"> | ||
<Grid> | ||
<TextBlock HorizontalAlignment="Center" | ||
TextWrapping="Wrap" | ||
Text="This is the expanded content" | ||
VerticalAlignment="Center" | ||
Style="{StaticResource HeaderTextBlockStyle}" /> | ||
TextWrapping="Wrap" | ||
Text="This is the expanded content" | ||
VerticalAlignment="Center" /> | ||
</Grid> | ||
|
||
<controls:Expander.ContentOverlay> | ||
<Grid Background="Red" MinHeight="250"> | ||
<Grid Height="120"> | ||
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. We should stick with |
||
<TextBlock Text="This is the content overlay" | ||
HorizontalAlignment="Center" | ||
VerticalAlignment="Center" /> | ||
HorizontalAlignment="Center" | ||
VerticalAlignment="Center" /> | ||
</Grid> | ||
</controls:Expander.ContentOverlay> | ||
</controls:Expander> | ||
|
||
<controls:Expander x:Name="Expander2" VerticalAlignment="Top" Margin="0" | ||
Header="This is the header - expander 2" | ||
HorizontalContentAlignment="Stretch" | ||
Foreground="White" | ||
Background="@[BackgroundExpander2:Brush:Black]" | ||
IsExpanded="@[IsExpanded2:Bool:True]@" | ||
ExpandDirection="@[ExpandDirection2:Enum:ExpandDirection.Right]"> | ||
<Grid Height="250"> | ||
Header="This is the header - expander 2" | ||
HorizontalContentAlignment="Stretch" | ||
IsExpanded="@[IsExpanded2:Bool:True]@" | ||
ExpandDirection="@[ExpandDirection2:Enum:ExpandDirection.Right]"> | ||
<Grid Height="256" | ||
Background="{ThemeResource SystemControlBackgroundBaseHighBrush}"> | ||
<TextBlock HorizontalAlignment="Center" | ||
TextWrapping="Wrap" | ||
Text="This is the expanded content without a content overlay" | ||
VerticalAlignment="Center" | ||
Style="{StaticResource HeaderTextBlockStyle}" /> | ||
TextWrapping="Wrap" | ||
Text="This is the expanded content without a content overlay" | ||
VerticalAlignment="Center" | ||
Foreground="{ThemeResource SystemControlForegroundChromeLowBrush}" /> | ||
</Grid> | ||
</controls:Expander> | ||
</StackPanel> | ||
|
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.
The expander no longer fully collapses. For example, change the second expander direction to
Down
and collapse it. The expander should completely collapse into the header.The background property should also change the background of the header when collapsed.
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.
Issue 1 is fixed, but issue 2 conflicts with the design redlines a little bit. I did my best to compensate though. Take a look and let me know what you think.