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

[Spaces] Remove “identifier” column from Spaces listing #201290

Merged
merged 9 commits into from
Nov 27, 2024

Conversation

tsullivan
Copy link
Member

@tsullivan tsullivan commented Nov 21, 2024

Summary

Addresses: https://github.com/elastic/kibana-team/issues/1241?reload=1#top
Closes: #195590

Changes

  1. Remove the "identifier" column from the Spaces grid listing
    1. Add a readonly text field to the Space edit form to allow users to see the identifier
  2. Remove granular width adjustments from the "Description" and "Solution view" columns to avoid header titles from showing ellipses

Screenshots

Spaces listing, before:
image

Spaces listing, after:
image

Spaces edit form, before
image

Spaces edit form, after
image

@tsullivan tsullivan marked this pull request as ready for review November 21, 2024 23:26
@tsullivan tsullivan requested a review from a team as a code owner November 21, 2024 23:26
@tsullivan tsullivan added Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! release_note:skip Skip the PR/issue when compiling release notes Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Nov 21, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-security (Team:Security)

@elasticmachine
Copy link
Contributor

Pinging @elastic/appex-sharedux (Team:SharedUX)

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #17 / aiops log pattern analysis attaches log pattern analysis table to a case

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
spaces 257.0KB 256.9KB -167.0B

History

@tsullivan tsullivan merged commit 7a2edb4 into elastic:main Nov 27, 2024
23 checks passed
@tsullivan tsullivan deleted the spaces-mgmt/remove-id-col branch November 27, 2024 00:03
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12041114985

@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.x Backport failed because of merge conflicts

You might need to backport the following PRs to 8.x:
- [Lens] Embeddable react refactor (#186642)
- Remove the callout for upgrading to get enterprise-level features (#200283)
- [Spaces UI] Role Editor Flyout Should Match in Roles Mgmt (#198182)
- Removes visible features column from spaces grid when in serverless (#194438)

Manual backport

To create the backport manually run:

node scripts/backport --pr 201290

Questions ?

Please refer to the Backport tool documentation

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 28, 2024
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 201290 locally

4 similar comments
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 201290 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 201290 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 201290 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 201290 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 201290 locally

tsullivan added a commit to tsullivan/kibana that referenced this pull request Dec 5, 2024
## Summary

Addresses:
elastic/kibana-team#1241
Closes: elastic#195590

### Changes
1. Remove the "identifier" column from the Spaces grid listing
1. Add a readonly text field to the Space edit form to allow users to
see the identifier
2. Remove granular width adjustments from the "Description" and
"Solution view" columns to avoid header titles from showing ellipses

### Screenshots
**Spaces listing, before:**
<img width="2102" alt="image"
src="https://github.com/user-attachments/assets/7bc5a8d1-d9c2-4e03-af9b-1bf9025f2d70">

**Spaces listing, after:**
<img width="2102" alt="image"
src="https://github.com/user-attachments/assets/6f69b2cf-c381-43be-a6fd-e294f58b3f15">

**Spaces edit form, before**
<img width="2102" alt="image"
src="https://github.com/user-attachments/assets/a622695f-400c-436c-9cbd-c110d83d3a53">

**Spaces edit form, after**
<img width="2102" alt="image"
src="https://github.com/user-attachments/assets/d5f2fd3f-ec4b-49c8-aece-a5b10dec37b2">

(cherry picked from commit 7a2edb4)
@tsullivan
Copy link
Member Author

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

1 similar comment
@kibanamachine
Copy link
Contributor

Looks like this PR has a backport PR but it still hasn't been merged. Please merge it ASAP to keep the branches relatively in sync.

tsullivan added a commit that referenced this pull request Dec 9, 2024
… (#203164)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Spaces] Remove “identifier” column from Spaces listing
(#201290)](#201290)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Tim
Sullivan","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-27T00:03:49Z","message":"[Spaces]
Remove “identifier” column from Spaces listing (#201290)\n\n##
Summary\r\n\r\nAddresses:\r\nhttps://github.com/elastic/kibana-team/issues/1241?reload=1#top\r\nCloses:
https://github.com/elastic/kibana/issues/195590\r\n\r\n### Changes\r\n1.
Remove the \"identifier\" column from the Spaces grid listing\r\n1. Add
a readonly text field to the Space edit form to allow users to\r\nsee
the identifier\r\n2. Remove granular width adjustments from the
\"Description\" and\r\n\"Solution view\" columns to avoid header titles
from showing ellipses\r\n\r\n### Screenshots\r\n**Spaces listing,
before:**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7bc5a8d1-d9c2-4e03-af9b-1bf9025f2d70\">\r\n\r\n**Spaces
listing, after:**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/6f69b2cf-c381-43be-a6fd-e294f58b3f15\">\r\n\r\n**Spaces
edit form, before**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a622695f-400c-436c-9cbd-c110d83d3a53\">\r\n\r\n**Spaces
edit form, after**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/d5f2fd3f-ec4b-49c8-aece-a5b10dec37b2\">","sha":"7a2edb4811dad4aec91a7a195cd1e51cd5b49a32","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Security","release_note:skip","backport
missing","v9.0.0","Team:SharedUX","backport:prev-minor"],"number":201290,"url":"https://github.com/elastic/kibana/pull/201290","mergeCommit":{"message":"[Spaces]
Remove “identifier” column from Spaces listing (#201290)\n\n##
Summary\r\n\r\nAddresses:\r\nhttps://github.com/elastic/kibana-team/issues/1241?reload=1#top\r\nCloses:
https://github.com/elastic/kibana/issues/195590\r\n\r\n### Changes\r\n1.
Remove the \"identifier\" column from the Spaces grid listing\r\n1. Add
a readonly text field to the Space edit form to allow users to\r\nsee
the identifier\r\n2. Remove granular width adjustments from the
\"Description\" and\r\n\"Solution view\" columns to avoid header titles
from showing ellipses\r\n\r\n### Screenshots\r\n**Spaces listing,
before:**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7bc5a8d1-d9c2-4e03-af9b-1bf9025f2d70\">\r\n\r\n**Spaces
listing, after:**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/6f69b2cf-c381-43be-a6fd-e294f58b3f15\">\r\n\r\n**Spaces
edit form, before**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a622695f-400c-436c-9cbd-c110d83d3a53\">\r\n\r\n**Spaces
edit form, after**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/d5f2fd3f-ec4b-49c8-aece-a5b10dec37b2\">","sha":"7a2edb4811dad4aec91a7a195cd1e51cd5b49a32"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201290","number":201290,"mergeCommit":{"message":"[Spaces]
Remove “identifier” column from Spaces listing (#201290)\n\n##
Summary\r\n\r\nAddresses:\r\nhttps://github.com/elastic/kibana-team/issues/1241?reload=1#top\r\nCloses:
https://github.com/elastic/kibana/issues/195590\r\n\r\n### Changes\r\n1.
Remove the \"identifier\" column from the Spaces grid listing\r\n1. Add
a readonly text field to the Space edit form to allow users to\r\nsee
the identifier\r\n2. Remove granular width adjustments from the
\"Description\" and\r\n\"Solution view\" columns to avoid header titles
from showing ellipses\r\n\r\n### Screenshots\r\n**Spaces listing,
before:**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/7bc5a8d1-d9c2-4e03-af9b-1bf9025f2d70\">\r\n\r\n**Spaces
listing, after:**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/6f69b2cf-c381-43be-a6fd-e294f58b3f15\">\r\n\r\n**Spaces
edit form, before**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/a622695f-400c-436c-9cbd-c110d83d3a53\">\r\n\r\n**Spaces
edit form, after**\r\n<img width=\"2102\"
alt=\"image\"\r\nsrc=\"https://github.com/user-attachments/assets/d5f2fd3f-ec4b-49c8-aece-a5b10dec37b2\">","sha":"7a2edb4811dad4aec91a7a195cd1e51cd5b49a32"}}]}]
BACKPORT-->
@kibanamachine kibanamachine added v8.18.0 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Dec 9, 2024
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
## Summary

Addresses:
elastic/kibana-team#1241
Closes: elastic#195590

### Changes
1. Remove the "identifier" column from the Spaces grid listing
1. Add a readonly text field to the Space edit form to allow users to
see the identifier
2. Remove granular width adjustments from the "Description" and
"Solution view" columns to avoid header titles from showing ellipses

### Screenshots
**Spaces listing, before:**
<img width="2102" alt="image"
src="https://github.com/user-attachments/assets/7bc5a8d1-d9c2-4e03-af9b-1bf9025f2d70">

**Spaces listing, after:**
<img width="2102" alt="image"
src="https://github.com/user-attachments/assets/6f69b2cf-c381-43be-a6fd-e294f58b3f15">

**Spaces edit form, before**
<img width="2102" alt="image"
src="https://github.com/user-attachments/assets/a622695f-400c-436c-9cbd-c110d83d3a53">

**Spaces edit form, after**
<img width="2102" alt="image"
src="https://github.com/user-attachments/assets/d5f2fd3f-ec4b-49c8-aece-a5b10dec37b2">
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! Team:SharedUX Team label for AppEx-SharedUX (formerly Global Experience) v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Spaces] Spaces page table has lots of ellipsis
4 participants