Skip to content
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

Click on Expander header support #1291

Merged
merged 1 commit into from
Apr 12, 2014
Merged

Click on Expander header support #1291

merged 1 commit into from
Apr 12, 2014

Conversation

Dotnetifier
Copy link
Contributor

I noticed that the Expander control does not open when you click the header text, which is not the default behavior, and may be annoying.

I set the Header as the content of the ToggleButton that determines whether the Expander is open or not, which fixed the problem.

punker76 added a commit that referenced this pull request Apr 12, 2014
Click on Expander header support
@punker76 punker76 merged commit ca4d735 into MahApps:master Apr 12, 2014
@Dotnetifier Dotnetifier deleted the patch-1 branch April 13, 2014 09:15
michaelmairegger added a commit to michaelmairegger/MahApps.Metro that referenced this pull request Apr 14, 2014
@michaelmairegger
Copy link
Contributor

There was introduced an optical bug which I try to solve

Before this commit
before
After this commit
after

The XAML I use is as follows:

<Grid Name="Grid">
    <Grid.ColumnDefinitions>
        <ColumnDefinition />
        <ColumnDefinition Width="24" />
    </Grid.ColumnDefinitions>
    [...]
</Grid>

@Dotnetifier Dotnetifier restored the patch-1 branch April 14, 2014 07:40
@Dotnetifier Dotnetifier deleted the patch-1 branch April 14, 2014 07:41
@Dotnetifier
Copy link
Contributor Author

Looks like I had to test it a little bit more... thanks!

@michaelmairegger
Copy link
Contributor

But my commit is untill now not complete.

michaelmairegger added a commit to michaelmairegger/MahApps.Metro that referenced this pull request Apr 23, 2014
Fix optical bug intruduced in MahApps#1291 (reverted from commit 7b9a60f)
@michaelmairegger
Copy link
Contributor

@punker76 Can you undo this merge, because of the problem I have shown in the screenshots untill we have found a solution for the header problem?

@punker76
Copy link
Member

@xxMUROxx done

@punker76 punker76 mentioned this pull request Jun 19, 2015
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants