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

770 save action and ux improvements #802

Merged
merged 18 commits into from
Apr 2, 2025

Conversation

pdelboca
Copy link
Member

@pdelboca pdelboca commented Apr 1, 2025

Fixes #770

  • Implements small UX improve to display an animation when the user clicks a button.
  • Adds spacing and margins to 0 in the central widget to fix alignments.
  • Do not enforce an aggressive QWidget {background: white;} to enforce a light mode to avoid side effects (Like menu and statusBar being affected)
  • Adds a statusBar() with some initial messages to communicate to the user main actions (add, delete, rename, read, validate, etc)
  • Organize and document our style.qss file

Please make sure that all the checks pass. Please add here any additional information regarding this pull request. It's highly recommended that you link this PR to an issue (please create one if it doesn't exist for this PR)

Copy link

cloudflare-workers-and-pages bot commented Apr 1, 2025

Deploying opendataeditor with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6d6f329
Status: ✅  Deploy successful!
Preview URL: https://10d529f4.opendataeditor.pages.dev
Branch Preview URL: https://770-save-action-and-ux-impro.opendataeditor.pages.dev

View logs

@pdelboca pdelboca requested review from rustico and romicolman April 1, 2025 11:56
Copy link
Collaborator

@romicolman romicolman left a comment

Choose a reason for hiding this comment

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

Pato, I have tested everything on Mac and it works OK. Thanks!

@pdelboca pdelboca merged commit f2cf99d into main Apr 2, 2025
5 of 6 checks passed
@pdelboca pdelboca deleted the 770-save-action-and-ux-improvements branch April 2, 2025 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Save - Action
3 participants