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

[Backport 2.17] Fit and Finish UX Fixes #1095

Closed

Conversation

jowg-amazon
Copy link
Contributor

Backport 4a2b603 from #1092.

Issues Resolved

[List any issues this PR will resolve]

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

* remove horizontal line below search bar

Signed-off-by: Joanne Wang <jowg@amazon.com>

* space between header and panel 16px and space around table 16px

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix 16px around table and search bar

Signed-off-by: Joanne Wang <jowg@amazon.com>

* remove pagination from top of table

Signed-off-by: Joanne Wang <jowg@amazon.com>

* add button to create monitor

Signed-off-by: Joanne Wang <jowg@amazon.com>

* add tooltip to delete

Signed-off-by: Joanne Wang <jowg@amazon.com>

* remove space on trash icon

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix gutter size

Signed-off-by: Joanne Wang <jowg@amazon.com>

* move edit under actions

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix notifications breadcrumb redirect

Signed-off-by: Joanne Wang <jowg@amazon.com>

* spacing from title to panel 16 for view monitor

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix some padding and remove horiz rule for findings section

Signed-off-by: Joanne Wang <jowg@amazon.com>

* change update monitor to edit monitor

Signed-off-by: Joanne Wang <jowg@amazon.com>

* change update to save and fix padding for create monitor

Signed-off-by: Joanne Wang <jowg@amazon.com>

* change monitor details padding and fix link to new page in alerts card

Signed-off-by: Joanne Wang <jowg@amazon.com>

* remove boarder when empty state

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix search bar spacing above

Signed-off-by: Joanne Wang <jowg@amazon.com>

* make create and edit monitor panels 16px padding

Signed-off-by: Joanne Wang <jowg@amazon.com>

* rename

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix original tabs

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix spacing on original monitor page

Signed-off-by: Joanne Wang <jowg@amazon.com>

* add pagination back to the alerts page

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix spacing for alerts dashboard flyout

Signed-off-by: Joanne Wang <jowg@amazon.com>

* cleanup

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix unit tests and run yarn test:jest -u

Signed-off-by: Joanne Wang <jowg@amazon.com>

* fix cypress tests

Signed-off-by: Joanne Wang <jowg@amazon.com>

---------

Signed-off-by: Joanne Wang <jowg@amazon.com>
@jowg-amazon
Copy link
Contributor Author

acknowledge_alerts_modal_spec.js is a flaky test

@jowg-amazon
Copy link
Contributor Author

Closing in favor of #1094

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.

2 participants