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

[Hot Design] "Counter tutorial" feedback #38

Closed
NonameMissingNo opened this issue Dec 14, 2024 · 4 comments
Closed

[Hot Design] "Counter tutorial" feedback #38

NonameMissingNo opened this issue Dec 14, 2024 · 4 comments
Labels
difficulty/tbd kind/consumer-experience Categorizes issue or PR as related to improving the experience of consumers kind/documentation Categorizes an issue or PR as relevant to 3rd party dependencies that are consumed by this project tool/Hot Design™

Comments

@NonameMissingNo
Copy link

On which page?

https://platform.uno/docs/articles/studio/Hot%20Design/hot-design-getstarted-counter-tutorial.html?tabs=vs

Which tool(s) are affected?

Hot Design™

What's wrong?

Counter tutorial:
Downloading the SVG file should lead to a download site, not the actual svg (Some people would have trouble downloading that graphics)

Creating an app is explained already in Create an App inside Get Started

Signing in (or registering) is mentioned multiple times

Using static width and height might not be the best idea for a first app (It encourages non-responsive behaviours)

Minor nitpick: I personally prefer Flame icon, instead of Flame button

Any feedback?

No response

@NonameMissingNo NonameMissingNo added difficulty/tbd kind/consumer-experience Categorizes issue or PR as related to improving the experience of consumers kind/documentation Categorizes an issue or PR as relevant to 3rd party dependencies that are consumed by this project triage/untriaged Indicates an issue requires triaging or verification. labels Dec 14, 2024
@junelac
Copy link

junelac commented Dec 16, 2024

a simpler solution would be to write to do a "right click+save as" in the document

@NonameMissingNo
Copy link
Author

a simpler solution would be to write to do a "right click+save as" in the document

Also fair

@nickrandolph nickrandolph removed the triage/untriaged Indicates an issue requires triaging or verification. label Dec 17, 2024
@nickrandolph
Copy link

@NonameMissingNo thanks for the feedback.

@nickrandolph
Copy link

Downloading the SVG file should lead to a download site, not the actual svg (Some people would have trouble downloading that graphics)
NR: Added comment to right-click and save-as

Creating an app is explained already in Create an App inside Get Started
NR: It's a little repetitive but acceptable as some people will come to this doc first, without going to get started

Signing in (or registering) is mentioned multiple times
NR: Again, repetitive to get the point across

Using static width and height might not be the best idea for a first app (It encourages non-responsive behaviours)
NR: fixed width/height in context of an image is quite acceptable as it's rarely a good idea to auto-scale images. In the context of responsive apps, there's a bunch of strategies, auto-scale isn't a great one. We'll leave this as is, as it helps tidy the UI and provide pointers on setting property values (a key part of the walk through)

Minor nitpick: I personally prefer Flame icon, instead of Flame button
NR: Interesting feedback - from a consistency perspective we'll stick with button but I've made it consistent across the document

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
difficulty/tbd kind/consumer-experience Categorizes issue or PR as related to improving the experience of consumers kind/documentation Categorizes an issue or PR as relevant to 3rd party dependencies that are consumed by this project tool/Hot Design™
Projects
None yet
Development

No branches or pull requests

3 participants