reflex-app / reflex-app/reflex
Add design standard & implementation for alerts
Open
Nobody has claimed this yet.
Type: Enhancement
- Dominant language
- Vue
- Stars
- 34
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Description
There's currently no setup for customized alerts which will provide feedback to the user that something went wrong (preferably with specific next actions and details!). Because of this, there's still a lot of console.log() and alert() usage.
Examples of where alerts occur:
- Invalid URL entered in the URL bar (example: "my-url"). This currently just displays a default
alert(). - If a URL returns error "X-Frame-Origin". Currently there's no UI alert, just an error in the console.
- Trying to delete all of the artboards on the screen. There must be at least 1.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the existing console.log() and alert() usage, then inspect the URL bar, X-Frame-Origin error, and artboard deletion flows mentioned in the issue. Done means these cases use a consistent customized alert experience with useful details and next actions instead of the current default alerts or console-only errors.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- electron, javascript, nuxt
- Domain
- desktop, frontend
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100