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

Rounded Corners Sample #375

Open
1 of 3 tasks
stmoy opened this issue Mar 11, 2020 · 17 comments
Open
1 of 3 tasks

Rounded Corners Sample #375

stmoy opened this issue Mar 11, 2020 · 17 comments
Labels
enhancement Expand an existing sample or add feature to app new sample New feature or request

Comments

@stmoy
Copy link
Contributor

stmoy commented Mar 11, 2020

Is this a totally new sample, an addition to an existing sample page, or a modification of an existing sample page?

  • New sample
  • Addition to existing sample
  • Modification to existing sample

If this is an addition/modification to an existing sample page, which one?

Describe the sample

Add a new sample that shows off:

  1. Rounded corners on a set of existing controls
  2. How to customize rounded corners
  3. How to turn off rounded corners if necessary

Consider adding a link to the Xaml Fluent Theme Editor which also supports these scenarios.

Is your sample request related to a problem? Please describe it

Additional context

@Chips1234
Copy link
Contributor

I would like to work on this, just telling everyone

@stmoy
Copy link
Contributor Author

stmoy commented Jul 14, 2020

@Chips1234 - fantastic! Please reach out if you have any questions 😁

CC: @chigy @kikisaints for FYI

@Chips1234
Copy link
Contributor

Alright. Thanks. One problem is, how do I get rid of light-up when the textbox comes into focus? I put it into a border but the highlight is a problem (see picture)
Txt no radius

@chigy
Copy link
Member

chigy commented Jul 15, 2020

Alright. Thanks. One problem is, how do I get rid of light-up when the textbox comes into focus? I put it into a border but the highlight is a problem (see picture)
Txt no radius

@Chips1234 , could you tell me why you are trying to do that? We should just show rounding of the corner of the light-up and not start redesigning the control with this effort.

@Chips1234
Copy link
Contributor

I’m trying to put it into a border as shown here. I tried CornerRadius within the text box but an error says that “CornerRadius” does not support older versions of Win10.

@chigy
Copy link
Member

chigy commented Jul 15, 2020

@Chips1234 , I wanted to share how we do work for WinUI in terms of how each sample apps and documents are connected and we use them to help developers/designers. I thought this is important for you to be aware as you investigate the work you would work on this proposal.

This proposal touches on 3 ways we communicate with our developer/designer audiences:

  • Documentation - https://docs.microsoft.com/en-us/windows/uwp/design/style/rounded-corner
    • We document the guidance, sample code, etc. so that developer and designer audiences know how to use them properly. For this effort, we'd like to make sure the sample you write in Controls Gallery match that of sample shared in this doc. If you require to write additional code, we want to make sure we consider updating the guidance.
  • Controls Gallery - This is our "live" sharing of the controls and guidance.
    • Idea is that we are releasing examples and samples that match what we talk about in the above documentation. We also take screenshots of the samples to use them in the documentation so we want to make sure samples produce quality examples.
  • Theme Editor - This app let designer/developer to create a visual using the properties we expose to create types of UI customization that still ensure not deviating too much to require retemplating.
    • Idea is that the type of sample code you wrote in Controls Gallery are things users of Theme Editor can see updated automatically.

Let me know if you have any questions about this.

@chigy
Copy link
Member

chigy commented Jul 15, 2020

@kikisaints , please take a look and comment on @Chips1234 comment here?

I’m trying to put it into a border as shown here. I tried CornerRadius within the text box but an error says that “CornerRadius” does not support older versions of Win10.

@Chips1234
Copy link
Contributor

Chips1234 commented Jul 15, 2020

@stmoy So you want a separate page on corner radius or do you just want a new example in the existing pages (e.g. new example on the button page)?

@marcelwgn
Copy link
Collaborator

I think a separate page would be the best for this,

@Chips1234
Copy link
Contributor

Chips1234 commented Jul 15, 2020 via email

@Chips1234
Copy link
Contributor

@chigy If I set CornerRadius, I get this: Warning, XLS1105, The property ‘CornerRadius’ on type ‘Button’ is not available in the target platform minimum version (10.0.17134.0) supported by this project. I can set a conditional namespace though, is that what you want me to do?

@stmoy
Copy link
Contributor Author

stmoy commented Jul 16, 2020

I think a separate page would be the best for this,

+1

I can set a conditional namespace though, is that what you want me to do?

Conditional Xaml would be the appropriate way to handle this, yes. In addition, please be sure to set the MinimumUniversalAPIContract on the ControlExample.

@Chips1234
Copy link
Contributor

Chips1234 commented Jul 16, 2020 via email

@Chips1234 Chips1234 mentioned this issue Jul 24, 2020
3 tasks
@Poopooracoocoo
Copy link

a slider would be pretty cool!

btw is Fluent XAML Theme Editor (https://www.microsoft.com/store/productId/9N2XD3Q8X57C or https://github.com/microsoft/fluent-xaml-theme-editor) dead?

@stmoy
Copy link
Contributor Author

stmoy commented Aug 21, 2020

CC: @kikisaints for this:

btw is Fluent XAML Theme Editor (https://www.microsoft.com/store/productId/9N2XD3Q8X57C or https://github.com/microsoft/fluent-xaml-theme-editor) dead?

I believe the answer is "no, not dead - just not being actively invested in"

@marcelwgn marcelwgn added the enhancement Expand an existing sample or add feature to app label Jun 25, 2022
@Jay-o-Way
Copy link
Contributor

Is this still an issue with the latest version?

@marcelwgn
Copy link
Collaborator

@Jay-o-Way I'm pretty sure this is still relevant given there exists no sample that I know of that showcases this.

@karkarl karkarl added the new sample New feature or request label Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Expand an existing sample or add feature to app new sample New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants