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

Fix styling of table rows for deleted records #4833

Merged

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Jan 3, 2023

Summary

The current styling is brutal and not nice. Using the remove action hover style for deleted records in tables.

Before

deleted-records-before

After

deleted-records-after

Checklist

Check out our PR guidelines for more details.

The following are mandatory for all PRs:

  • I have written a thorough PR description.
  • I have kept my commits small and atomic.
  • I have used clear, explanatory commit messages.

The following are not always needed (cross them out if they are not):

  • I have added automated tests to cover my changes.
  • I have attached screenshots to demo visual changes.
  • I have opened a PR to update the guides.
  • I have updated the README to account for my changes.

@tvdeyen tvdeyen added changelog:solidus_backend Changes to the solidus_backend gem backport-v3.2 labels Jan 3, 2023
@tvdeyen tvdeyen requested a review from a team as a code owner January 3, 2023 16:44
The current styling is brutal and not nice. Using the
remove action hover style for deleted records in tables.
@tvdeyen tvdeyen force-pushed the fix-deleted-records-table-style branch from a2e4756 to 9788b19 Compare January 3, 2023 17:09
@codecov
Copy link

codecov bot commented Jan 3, 2023

Codecov Report

Merging #4833 (9788b19) into master (785995d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4833   +/-   ##
=======================================
  Coverage   86.16%   86.16%           
=======================================
  Files         578      578           
  Lines       14664    14664           
=======================================
  Hits        12635    12635           
  Misses       2029     2029           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

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

Thanks!

@kennyadsl kennyadsl merged commit 3e82378 into solidusio:master Jan 4, 2023
@github-actions
Copy link

github-actions bot commented Jan 4, 2023

💚 All backports created successfully

Status Branch Result
v3.2

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_backend Changes to the solidus_backend gem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants