Skip to content

Conversation

annadoesdesign
Copy link
Contributor

@annadoesdesign annadoesdesign commented May 5, 2025

  • minor update after search results page
  • added 8px gap
    Before:
Screenshot 2025-05-05 at 11 54 58 AM Screenshot 2025-05-05 at 11 55 02 AM

After
Screenshot 2025-05-05 at 11 54 45 AM

Screenshot 2025-05-05 at 11 54 51 AM Screenshot 2025-05-05 at 11 54 48 AM

@github-actions github-actions bot added the product PR or Issue related to the DataHub UI/UX label May 5, 2025
@datahub-cyborg datahub-cyborg bot added the needs-review Label for PRs that need review from a maintainer. label May 5, 2025
Copy link

codecov bot commented May 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

Copy link

alwaysmeticulous bot commented May 5, 2025

✅ Meticulous spotted visual differences in 110 of 1483 screens tested, but all differences have already been approved: view differences detected.

Meticulous evaluated ~10 hours of user flows against your PR.

Last updated for commit a25cdc7. This comment will update as new commits are pushed.

@datahub-cyborg datahub-cyborg bot added pending-submitter-merge and removed needs-review Label for PRs that need review from a maintainer. labels May 6, 2025
@asikowitz asikowitz merged commit 7a4232d into master May 7, 2025
33 checks passed
@asikowitz asikowitz deleted the ae--fix(ui)-impact-analysis-edit-download-buttons branch May 7, 2025 07:19
kartikey-visa pushed a commit to kartikey-visa/datahub that referenced this pull request Jul 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pending-submitter-merge product PR or Issue related to the DataHub UI/UX

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants