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

Rename Brush names #286

Merged
merged 49 commits into from Aug 31, 2023
Merged

Conversation

JanTamis
Copy link
Collaborator

@JanTamis
Copy link
Collaborator Author

@SKProCH Please let me know if I am renaming the correct properties

@SKProCH
Copy link
Collaborator

SKProCH commented Aug 28, 2023

I hadn't thought about renaming the property names, actually. However, let's keep the new names, I think it's the right thing to do. We should also rename ValidationErrorColor.

But the most important thing is to rename

And according to the renaming of these styles, also rename all their names in all style files (rename all old resources names across solution).

However, if you don't want to do that, fine, we can dump this PR as it is.

@JanTamis
Copy link
Collaborator Author

I am willing to rename all the styles 🙂. Just needed to know if I was on the right track.

@JanTamis JanTamis marked this pull request as ready for review August 30, 2023 19:24
@JanTamis
Copy link
Collaborator Author

@SKProCH Please let me know if there are areas I missed

@SKProCH
Copy link
Collaborator

SKProCH commented Aug 30, 2023

Seems like everything is fine.

But also I noticed that we can actually remove Hue from PrimaryHueLightBrush (and similar brushes). So it will be MaterialPrimaryLightBrush. What do you think?

@SKProCH
Copy link
Collaborator

SKProCH commented Aug 30, 2023

Also, can you please add a note with name changes?
Like:

  • Old -> New

This needs to be included to migration guide.

Also, if you can (and you want) it will be nice to have fallback brushes resources with old names. Just a resource dictionary with old names, which will be referencing new brushes (or new brushes colors)

@JanTamis
Copy link
Collaborator Author

Seems like everything is fine.

But also I noticed that we can actually remove Hue from PrimaryHueLightBrush (and similar brushes). So it will be MaterialPrimaryLightBrush. What do you think?

I think that that is a great idea

@JanTamis
Copy link
Collaborator Author

Also, can you please add a note with name changes? Like:

  • Old -> New

This needs to be included to migration guide.

Also, if you can (and you want) it will be nice to have fallback brushes resources with old names. Just a resource dictionary with old names, which will be referencing new brushes (or new brushes colors)

Where would you like me to add the notes to?

@SKProCH
Copy link
Collaborator

SKProCH commented Aug 31, 2023

Also, can you please add a note with name changes? Like:

  • Old -> New

This needs to be included to migration guide.
Also, if you can (and you want) it will be nice to have fallback brushes resources with old names. Just a resource dictionary with old names, which will be referencing new brushes (or new brushes colors)

Where would you like me to add the notes to?

Just paste it here

@JanTamis
Copy link
Collaborator Author

Also, can you please add a note with name changes? Like:

  • Old -> New

This needs to be included to migration guide.

Also, if you can (and you want) it will be nice to have fallback brushes resources with old names. Just a resource dictionary with old names, which will be referencing new brushes (or new brushes colors)

I unfortunately do not know how to add fallback brushes 😅

@JanTamis
Copy link
Collaborator Author

Also, can you please add a note with name changes? Like:

  • Old -> New

This needs to be included to migration guide.
Also, if you can (and you want) it will be nice to have fallback brushes resources with old names. Just a resource dictionary with old names, which will be referencing new brushes (or new brushes colors)

Where would you like me to add the notes to?

Just paste it here

okay

@SKProCH
Copy link
Collaborator

SKProCH commented Aug 31, 2023

Also, can you please add a note with name changes? Like:

  • Old -> New

This needs to be included to migration guide.

Also, if you can (and you want) it will be nice to have fallback brushes resources with old names. Just a resource dictionary with old names, which will be referencing new brushes (or new brushes colors)

I unfortunately do not know how to add fallback brushes 😅

Okay, no problem, I'll try to do it myself, after PR merge

@JanTamis
Copy link
Collaborator Author

JanTamis commented Aug 31, 2023

Brushes

  • PrimaryHueLightBrush -> MaterialPrimaryLightBrush
  • PrimaryHueLightForegroundBrush -> MaterialPrimaryLightForegroundBrush
  • PrimaryHueMidBrush -> MaterialPrimaryMidBrush
  • PrimaryHueMidForegroundBrush -> MaterialPrimaryMidForegroundBrush
  • PrimaryHueDarkBrush -> MaterialPrimaryDarkBrush
  • PrimaryHueDarkForegroundBrush -> MaterialPrimaryForegroundBrush
  • SecondaryHueLightBrush -> MaterialSecondaryLightBrush
  • SecondaryHueLightForegroundBrush -> MaterialSecondaryLightForegroundBrush
  • SecondaryHueMidBrush -> MaterialSecondaryMidBrush
  • SecondaryHueDarkBrush -> MaterialSecondaryDarkBrush
  • SecondaryHueDarkForegroundBrush -> MaterialSecondaryDarkForegroundBrush
  • ValidationErrorBrush -> MaterialValidationErrorBrush
  • MaterialDesignBackground -> MaterialBackgroundBrush
  • MaterialDesignPaper -> MaterialPaperBrush
  • MaterialDesignCardBackground -> MaterialCardBackgroundBrush
  • MaterialDesignToolBarBackground -> MaterialToolBarBackgroundBrush
  • MaterialDesignBody -> MaterialBodyBrush
  • MaterialDesignBodyLight -> MaterialBodyLightBrush
  • MaterialDesignColumnHeader -> MaterialColumnHeaderBrush
  • MaterialDesignCheckBoxOff -> MaterialCheckBoxOffBrush
  • MaterialDesignCheckBoxDisabled -> MaterialCheckBoxDisabledBrush
  • MaterialDesignTextBoxBorder -> MaterialTextBoxBorderBrush
  • MaterialDesignDivider -> MaterialDividerBrush
  • MaterialDesignSelection -> MaterialSelectionBrush
  • MaterialDesignToolForeground -> MaterialToolForegroundBrush
  • MaterialDesignToolBackground -> MaterialToolBackgroundBrush
  • MaterialDesignFlatButtonClick -> MaterialFlatButtonClickBrush
  • MaterialDesignFlatButtonRipple -> MaterialFlatButtonRippleBrush
  • MaterialDesignToolTipBackground -> MaterialToolTipBackgroundBrush
  • MaterialDesignChipBackground -> MaterialChipBackgroundBrush
  • MaterialDesignSnackbarBackground -> MaterialSnackbarBackgroundBrush
  • MaterialDesignSnackbarMouseOver -> MaterialSnackbarMouseOverBrush
  • MaterialDesignSnackbarRipple -> MaterialSnackbarRippleBrush
  • MaterialDesignTextFieldBoxBackground -> MaterialTextFieldBoxBackgroundBrush
  • MaterialDesignTextFieldBoxHoverBackground -> MaterialTextFieldBoxHoverBackgroundBrush
  • MaterialDesignTextFieldBoxDisabledBackground -> MaterialTextFieldBoxDisabledBackgroundBrush
  • MaterialDesignTextAreaBorder -> MaterialTextAreaBorderBrush
  • MaterialDesignTextAreaInactiveBorder -> MaterialTextAreaInactiveBorderBrush
  • MaterialDesignDataGridRowHoverBackground -> MaterialDataGridRowHoverBackgroundBrush

Colors

  • ValidationErrorColor -> MaterialValidationErrorColor
  • MaterialDesignBackground -> MaterialBackgroundColor
  • MaterialDesignPaper -> MaterialPaperColor
  • MaterialDesignCardBackground -> MaterialCardBackgroundColor
  • MaterialDesignToolBarBackground -> MaterialToolBarBackgroundColor
  • MaterialDesignBody -> MaterialBodyColor
  • MaterialDesignBodyLight -> MaterialBodyLightColor
  • MaterialDesignColumnHeader -> MaterialColumnHeaderColor
  • MaterialDesignCheckBoxOff -> MaterialCheckBoxOffColor
  • MaterialDesignCheckBoxDisabled -> MaterialCheckBoxDisabledColor
  • MaterialDesignTextBoxBorder -> MaterialTextBoxBorderColor
  • MaterialDesignDivider -> MaterialDividerColor
  • MaterialDesignSelection -> MaterialSelectionColor
  • MaterialDesignToolForeground -> MaterialToolForegroundColor
  • MaterialDesignToolBackground -> MaterialToolBackgroundColor
  • MaterialDesignFlatButtonClick -> MaterialFlatButtonClickColor
  • MaterialDesignFlatButtonRipple -> MaterialFlatButtonRippleColor
  • MaterialDesignToolTipBackground -> MaterialToolTipBackgroundColor
  • MaterialDesignChipBackground -> MaterialChipBackgroundColor
  • MaterialDesignSnackbarBackground -> MaterialSnackbarBackgroundColor
  • MaterialDesignSnackbarMouseOver -> MaterialSnackbarMouseOverColor
  • MaterialDesignSnackbarRipple -> MaterialSnackbarRippleColor
  • MaterialDesignTextFieldBoxBackground -> MaterialTextFieldBoxBackgroundColor
  • MaterialDesignTextFieldBoxHoverBackground -> MaterialTextFieldBoxHoverBackgroundColor
  • MaterialDesignTextFieldBoxDisabledBackground -> MaterialTextFieldBoxDisabledBackgroundColor
  • MaterialDesignTextAreaBorder -> MaterialTextAreaBorderColor
  • MaterialDesignTextAreaInactiveBorder -> MaterialTextAreaInactiveBorderColor
  • MaterialDesignDataGridRowHoverBackground -> MaterialDataGridRowHoverBackgroundColor

@SKProCH
Copy link
Collaborator

SKProCH commented Aug 31, 2023

Yep, seems like everything is fine, thanks. Should I merge right now?

@JanTamis
Copy link
Collaborator Author

yes, your welcome

@SKProCH SKProCH merged commit c1e4241 into AvaloniaCommunity:master Aug 31, 2023
@JanTamis JanTamis deleted the 3.0_brush_rename branch August 31, 2023 18:09
@SKProCH
Copy link
Collaborator

SKProCH commented Aug 31, 2023

Thanks. I actually can give write permissions for the repo if you want to contribute further.

@JanTamis
Copy link
Collaborator Author

That would be useful.

@SKProCH
Copy link
Collaborator

SKProCH commented Aug 31, 2023

Invited you. Did you have any messenger such as telegram or discord, etc?

@JanTamis
Copy link
Collaborator Author

JanTamis commented Aug 31, 2023

I have discord Jan_Tamis#4394

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.

None yet

2 participants