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

Update InfoBarSamplePage.xaml with more Infos and color as well added… #1027

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AryanParashar24
Copy link

@AryanParashar24 AryanParashar24 commented Oct 6, 2023

… to them depending onto the different conditions as founded in the code

two more Infos are been added in the code with the colors as well been added inot the code specifying the conditions been faced by the customer or developer based on the status of their sitaution

GitHub Issue (If applicable): #

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting)
  • Refactoring (no functional changes, no api changes)
  • Build or CI related changes
  • Documentation content changes
  • Project automation
  • Other... Please describe:

What is the current behavior?

What is the new behavior?

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tested on iOS.
  • Tested on Wasm.
  • Tested on Android.
  • Tested on UWP.
  • Tested in both Light and Dark themes.
  • Associated with an issue (GitHub or internal)

Other information

Internal Issue (If applicable):

… to them depending onto the different conditions as founded in the code

two more Infos are been added in the code with the colors as well been added inot the code specifying the conditions been faced by the customer or developer based on the status of their sitaution
@morning4coffe-dev
Copy link
Member

@AryanParashar24 Tests seems to be failing on SamplePages\InfoBarSamplePage.xaml(125,11): Error UXAML0001: The 'winui:InfoBar' start tag on line 115 position 9 does not match the end tag of 'winui:InfoBar.Content'. Can you look into it please?

@morning4coffe-dev
Copy link
Member

Also @MartinZikmund isn't this one related to #308? 🤔

Comment on lines 123 to 125
<Button Style="{StaticResource InformationButtonStyle}"
Content="DISMISS" />
</winui:InfoBar.Content>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing the winui:InfoBar.Content starting tag

@CLAassistant
Copy link

CLAassistant commented Oct 17, 2023

CLA assistant check
All committers have signed the CLA.

'winui:InfoBar' start tag has been added in to the code which was missing from the file hope it solves out the issue now
@AryanParashar24
Copy link
Author

Hey @morning4coffe-dev @mrlacey plz le me know if it could be improved a bit more by changing anything else

<Setter Property="Background" Value="Yellow" />
<Setter Property="Foreground" Value="Black" />
</Style>
<!-- Add more styles as needed for different severities -->
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"severities" or "scenarios"?

<StackPanel Spacing="20"
Margin="0,20,0,0">
<Page.Resources>
<!-- Button Styles -->
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is anything lost by not basing these new styles on TextButtonStyle?
Such as Min sizes and different focus visuals?
Is this intentional/deliberate? All you've done is change the styles to use different colors and with no explanation of why.

@mrlacey
Copy link

mrlacey commented Oct 17, 2023

Hey @morning4coffe-dev @mrlacey plz le me know if it could be improved a bit more by changing anything else

Not sure if you meant me or @MartinZikmund but I have added some comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants