Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs(Window): remove centered parameter #2826

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Tsvetomir-Hr
Copy link
Contributor

@Tsvetomir-Hr Tsvetomir-Hr self-assigned this Mar 12, 2025
@Tsvetomir-Hr Tsvetomir-Hr requested review from a team as code owners March 12, 2025 12:16
Copy link
Contributor

@dimodi dimodi left a comment

Choose a reason for hiding this comment

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

  • Explain how to center the Window initially by not using Top and Left at all, or y setting them to string.Empty.
  • Enhance the "Top and Left" example with the button that centers the Window back.

@Tsvetomir-Hr
Copy link
Contributor Author

  • Explain how to center the Window initially by not using Top and Left at all*

For this, I can add at the beginning of the article that if these parameters are not defined the Window will be in the center. Is that good for you @dimodi ?

  • Enhance the "Top and Left" example with the button that centers the Window back.*

If I update the example to center the window through a button what will be the purpose of the KB? #2821

@Tsvetomir-Hr Tsvetomir-Hr requested a review from dimodi March 13, 2025 11:45
@Tsvetomir-Hr Tsvetomir-Hr requested a review from dimodi March 18, 2025 15:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants