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 #1094

Merged
merged 1 commit into from
Sep 17, 2024
Merged

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 4a2b603 from #1092.

* remove horizontal line below search bar

Signed-off-by: Joanne Wang <[email protected]>

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

Signed-off-by: Joanne Wang <[email protected]>

* fix 16px around table and search bar

Signed-off-by: Joanne Wang <[email protected]>

* remove pagination from top of table

Signed-off-by: Joanne Wang <[email protected]>

* add button to create monitor

Signed-off-by: Joanne Wang <[email protected]>

* add tooltip to delete

Signed-off-by: Joanne Wang <[email protected]>

* remove space on trash icon

Signed-off-by: Joanne Wang <[email protected]>

* fix gutter size

Signed-off-by: Joanne Wang <[email protected]>

* move edit under actions

Signed-off-by: Joanne Wang <[email protected]>

* fix notifications breadcrumb redirect

Signed-off-by: Joanne Wang <[email protected]>

* spacing from title to panel 16 for view monitor

Signed-off-by: Joanne Wang <[email protected]>

* fix some padding and remove horiz rule for findings section

Signed-off-by: Joanne Wang <[email protected]>

* change update monitor to edit monitor

Signed-off-by: Joanne Wang <[email protected]>

* change update to save and fix padding for create monitor

Signed-off-by: Joanne Wang <[email protected]>

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

Signed-off-by: Joanne Wang <[email protected]>

* remove boarder when empty state

Signed-off-by: Joanne Wang <[email protected]>

* fix search bar spacing above

Signed-off-by: Joanne Wang <[email protected]>

* make create and edit monitor panels 16px padding

Signed-off-by: Joanne Wang <[email protected]>

* rename

Signed-off-by: Joanne Wang <[email protected]>

* fix original tabs

Signed-off-by: Joanne Wang <[email protected]>

* fix spacing on original monitor page

Signed-off-by: Joanne Wang <[email protected]>

* add pagination back to the alerts page

Signed-off-by: Joanne Wang <[email protected]>

* fix spacing for alerts dashboard flyout

Signed-off-by: Joanne Wang <[email protected]>

* cleanup

Signed-off-by: Joanne Wang <[email protected]>

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

Signed-off-by: Joanne Wang <[email protected]>

* fix cypress tests

Signed-off-by: Joanne Wang <[email protected]>

---------

Signed-off-by: Joanne Wang <[email protected]>
(cherry picked from commit 4a2b603)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@AWSHurneyt AWSHurneyt merged commit 51d2c95 into 2.17 Sep 17, 2024
12 of 15 checks passed
AWSHurneyt added a commit to AWSHurneyt/alerting-dashboards-plugin that referenced this pull request Sep 24, 2024
@AWSHurneyt AWSHurneyt mentioned this pull request Sep 24, 2024
5 tasks
AWSHurneyt added a commit to AWSHurneyt/alerting-dashboards-plugin that referenced this pull request Sep 24, 2024
jowg-amazon added a commit to jowg-amazon/alerting-dashboards-plugin that referenced this pull request Sep 24, 2024
jowg-amazon added a commit to jowg-amazon/alerting-dashboards-plugin that referenced this pull request Sep 24, 2024
AWSHurneyt pushed a commit that referenced this pull request Sep 24, 2024
* Revert "Fit and Finish UX changes Pt 2 (#1099) (#1107)"

This reverts commit bfb0d79.

Signed-off-by: Joanne Wang <[email protected]>

* Revert "Fit and Finish UX Fixes (#1092) (#1094)"

This reverts commit 51d2c95.

Signed-off-by: Joanne Wang <[email protected]>

---------

Signed-off-by: Joanne Wang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants