Skip to content

Fix long agent policy name accessibility in add/edit integration#113828

Merged
zizhouW merged 1 commit intoelastic:masterfrom
zizhouW:issue-113820-long-agent-policy-name
Oct 6, 2021
Merged

Fix long agent policy name accessibility in add/edit integration#113828
zizhouW merged 1 commit intoelastic:masterfrom
zizhouW:issue-113820-long-agent-policy-name

Conversation

@zizhouW
Copy link

@zizhouW zizhouW commented Oct 4, 2021

Summary

Fixes #113820 where long agent policy name pushes page title to the very left when on add/edit integrations page

image

Checklist

Delete any items that are not applicable to this PR.

@zizhouW zizhouW requested a review from a team as a code owner October 4, 2021 18:27
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Oct 4, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@zizhouW zizhouW added release_note:skip Skip the PR/issue when compiling release notes v7.16.0 labels Oct 4, 2021
@juliaElastic
Copy link
Contributor

should we add auto-backport and v7.15.1 labels?

@juliaElastic
Copy link
Contributor

@zizhouW hi, could you check the mobile breakpoint as well? looks a bit weird aligned to the left (while Agent policy text aligned to the right), and there is more space for the name.

image

previously:
https://release-oblt.elastic.dev/app/fleet/integrations/apache-0.8.1/add-integration?policyId=a2b32670-25b9-11ec-86f8-9580d304e63b
image

@zizhouW zizhouW added auto-backport Deprecated - use backport:version if exact versions are needed v7.15.1 labels Oct 5, 2021
@zizhouW
Copy link
Author

zizhouW commented Oct 5, 2021

@juliaElastic addressed the comment above. Thanks for pointing it out!

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
fleet 598.2KB 598.4KB +211.0B

History

  • 💚 Build #157728 succeeded 8ae65ebb30a024de95927de6ea68aa5e7f7d0350
  • 💔 Build #157721 failed d7d43632055174057ffa4e8fa3598b8f3604ea6f

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@juliaElastic
Copy link
Contributor

looks good now
image

@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x
7.15

The backport PRs will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Oct 6, 2021
kibanamachine added a commit that referenced this pull request Oct 6, 2021
@amolnater-qasource
Copy link

Hi @jen-huang
We have revalidated this issue and found it fixed on 7.15.1 BC-1.

  • Long agent policy name doesn't push page title to the left on add/edit integrations page.

Build details:
BUILD: 44185
COMMIT: c1aa1ef

Screenshot:
13
14

cc: @EricDavisX
Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v7.15.1 v7.16.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Long agent policy name in integrations pushes Title to the side

8 participants