Turn off Notification Center from Preferences Panel.

[QilongTang]

Please Note:

1. Before submitting the PR, please review How to Contribute to Dynamo
2. Dynamo Team will meet 1x a month to review PRs found on Github (Issues will be handled separately)
3. PRs will be reviewed from oldest to newest
4. If a reviewed PR requires changes by the owner, the owner of the PR has 30 days to respond. If the PR has seen no activity by the next session, it will be either closed by the team or depending on its utility will be taken over by someone on the team
5. PRs should use either Dynamo's default PR template or one of these other template options in order to be considered for review.
6. PRs that do not have one of the Dynamo PR templates completely filled out with all declarations satisfied will not be reviewed by the Dynamo team.
7. PRs made to the DynamoRevit repo will need to be cherry-picked into all the DynamoRevit Release branches that Dynamo supports. Contributors will be responsible for cherry-picking their reviewed commits to the other branches after a LGTM label is added to the PR.

Purpose

Per https://jira.autodesk.com/browse/DYN-5177, New setting entry was added in Dynamo setting: EnableNotificationCenter

• Notification Center now can be disabled in the Preferences panel
• The Notification bell has been moved to the right side of the export menu
• The notification bell got a new look
• Updated unit tests also to include serializing and deserializing the new setting

Declarations

Check these if you believe they are true

• The codebase is in a better state after this PR
• Is documented according to the standards
• The level of testing this PR includes is appropriate
• User facing strings, if any, are extracted into *.resx files
• All tests pass using the self-service CI.
• Snapshot of UI changes, if any.
• Changes to the API follow Semantic Versioning and are documented in the API Changes document.
• This PR modifies some build requirements and the readme is updated

Release Notes

Users can turn off Notification Center from the Preferences Panel.

Reviewers

@DynamoDS/dynamo

FYIs

(FILL ME IN, Optional) Names of anyone else you wish to be notified of

[Jingyi-Wen]

Hi @QilongTang , just a suggestion - can we change the text for the toggle to "Receive notification"? Since the header already says Notification Center, it seems a bit redundant.

[QilongTang]

eems

Will do

[zeusongit]

why are we removing these setters here? Are these not used anymore?

[QilongTang]

@zeusongit Only removing the private value setter since these private values are not used. Suggested by VS

[RobertGlobant20]

LGTM

[QilongTang]

@RobertGlobant20 @filipeotero Any idea how to change the hover-over background of the notification bell?

[RobertGlobant20]

@RobertGlobant20 @filipeotero Any idea how to change the hover-over background of the notification bell?

@QilongTang I think you need to define a Style in DynamoModern.xaml file (or in the same xaml file, ShortcutToolbar.xaml), at the end the FontAwesome behaves in a similar way than any wpf control.

and then apply the colors for foreground, background using ControlTemplate.Triggers, something like this:

Let me know if it works. Thanks

[QilongTang]

@RobertGlobant20 This is what I am currently adding, but it keep crashing Dynamo while build is fine:

    <Style x:Key="NotificationBellButtonStyle" TargetType="{x:Type fa:FontAwesome}">
        <Setter Property="Height" Value="Auto" />
        <Setter Property="Template">
            <Setter.Value>
                <ControlTemplate TargetType="{x:Type MenuItem}">
                    <ControlTemplate.Triggers>
                        <Trigger Property="IsMouseOver" Value="True">
                            <Setter TargetName="ContentPresenter" Property="Background" Value="{StaticResource NodeContextMenuBackgroundHighlight}" />
                        </Trigger>
                        <Trigger Property="IsMouseOver" Value="False">
                            <Setter TargetName="ContentPresenter" Property="Background" Value="{StaticResource NodeContextMenuBackground}" />
                        </Trigger>
                    </ControlTemplate.Triggers>
                </ControlTemplate>
            </Setter.Value>
        </Setter>
    </Style>

                                    <fa:FontAwesome Margin="0,0,25,0"  
                                                    DockPanel.Dock="Top" 
                                                    Icon="{Binding Path=DataContext.PreferenceSettings.EnableNotificationCenter, RelativeSource={RelativeSource AncestorType={x:Type Window}}, 
                                                         Converter={StaticResource BoolToFAIconNameConverter}}"
                                                    Style="{StaticResource NotificationBellButtonStyle}"
                                                    HorizontalAlignment="Left"
                                                    VerticalAlignment="Center"
                                                    Foreground="{StaticResource PrimaryCharcoal300Brush}"
                                                    FontSize="18px"/>

[RobertGlobant20]

@QilongTang probably the crash is happening due that you are using ContentPresenter, I tried to use your style but it was not compiling because of the word "Template".
But the next Style worked for me (let me know if it helps):

[QilongTang]

@RobertGlobant20 Thanks! If I just apply the style on fontawesome, it works while only on the bell itself. Since the whole Notification Bell + message count is a button, so I had to apply the style to the button which does not work :( I believe that is what I tried before but not sure if I implemented anything wrong.

    <Style x:Key="NotificationBellStyle" TargetType="{x:Type Button}">
        <Style.Triggers>
            <Trigger Property="IsMouseOver" Value="True">
                <Setter Property="Background" Value="White" />
            </Trigger>
        </Style.Triggers>
    </Style>

[QilongTang]

@RobertGlobant20 Discussed with team, will merge first and follow up with you