Skip to content

feat: improve reset all settings button UI#649

Open
Nisha-Sanap wants to merge 3 commits into
magic-peach:mainfrom
Nisha-Sanap:improve-reset-settings-ui
Open

feat: improve reset all settings button UI#649
Nisha-Sanap wants to merge 3 commits into
magic-peach:mainfrom
Nisha-Sanap:improve-reset-settings-ui

Conversation

@Nisha-Sanap
Copy link
Copy Markdown
Contributor

Description

Improved the "Reset All Settings" UI by converting it into a proper destructive action button.

Changes Made

  • Converted reset text into a proper button
  • Added red destructive styling
  • Added hover, focus, and active states
  • Improved padding, border radius, and spacing
  • Added reset icon for better UX
  • Maintained existing reset functionality

Issue Fixed

Closes #646

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

@Nisha-Sanap is attempting to deploy a commit to the magic-peach1's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

✅ PR Format Check Passed — @Nisha-Sanap

Basic format checks passed. A maintainer will review your code changes.

This does not mean the PR is approved — it just means the format is correct.

@github-actions github-actions Bot added level:intermediate Intermediate level - 35 pts type:bug Bug fix type:design UI/UX design labels May 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

👋 Thanks for your PR, @Nisha-Sanap!

Welcome to Reframe — a browser-based video editor built for everyone 🎬

What happens next

  1. 🤖 Automated checks — build & TypeScript typecheck will run automatically
  2. Vercel preview — a preview deployment will be created (requires maintainer authorization for fork PRs)
  3. 👀 Code review — a maintainer will review your changes
  4. 🚀 Merge — once approved, your PR will be merged!

Quick checklist

  • PR title follows Conventional Commits (e.g. feat: add dark mode)
  • Linked the issue this PR closes (e.g. Closes #123)
  • Tested the changes locally (bun run dev)
  • Build passes (bun run build)

Useful links

Happy coding! 🎉

@github-actions github-actions Bot added the level:beginner Beginner level - 20 pts label May 18, 2026
@Nisha-Sanap Nisha-Sanap changed the title Improve "Reset All Settings" UI with proper button styling feat: improve reset all settings button UI May 18, 2026
@magic-peach
Copy link
Copy Markdown
Owner

Hey @Nisha-Sanap! 👋

We've added a new requirement for all PRs: a screen recording showing your changes working on your local machine must be attached before a PR can be merged.

Please add a recording to this PR that shows:

  1. bun run dev running at http://localhost:3000
  2. The full working flow of your change (demonstrate the feature/fix end-to-end)
  3. Any tests passing — if your change touches logic with tests, show bun run lint and bunx tsc --noEmit completing without errors in the terminal

How to record:

  • macOS: Cmd + Shift + 5 → Record Selected Portion, or QuickTime Player
  • Windows: Win + G → Xbox Game Bar → Capture
  • Linux: OBS Studio, GNOME Screenshot, or kazam
  • Any OS: Loom (free, easy to share)

Once you have the recording, drag the file directly into a comment on this PR, or paste a Loom link. This is now a hard requirement — see CONTRIBUTING.md for full details.

Thanks for contributing to Reframe! 🎬

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

level:beginner Beginner level - 20 pts level:intermediate Intermediate level - 35 pts type:bug Bug fix type:design UI/UX design

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Improve "Reset All Settings" UI - Make it a proper button

2 participants