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

[List] - renamed list items are still present in filtered lists #10204

Open
2 of 6 tasks
sagewall opened this issue Sep 3, 2024 · 4 comments
Open
2 of 6 tasks

[List] - renamed list items are still present in filtered lists #10204

sagewall opened this issue Sep 3, 2024 · 4 comments
Labels
0 - new New issues that need assignment. ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. calcite-components Issues specific to the @esri/calcite-components package. enhancement Issues tied to a new feature or request. estimate - 3 A day or two of work, likely requires updates to tests. has workaround Issues have a workaround available in the meantime. impact - p2 - want for an upcoming milestone User set priority impact status of p2 - want for an upcoming milestone needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. p - medium Issue is non core or affecting less that 60% of people using the library

Comments

@sagewall
Copy link

sagewall commented Sep 3, 2024

Check existing issues

Actual Behavior

If a list-item's label, value, or description properties are modified after a list is filtered, the renamed list item is still present in the list.

rename_filter_text

Expected Behavior

The list item who's properties were modified and no longer are matched by the filterText are filtered from the list.

Reproduction Sample

https://codepen.io/sagewall/pen/LYKrKWY

Reproduction Steps

  1. Open the codepen
  2. Enter some text in the filter input. "water" works well.
  3. Click on the action button to set new list item property values.
  4. Using your browsers developers tools, inspect the renamed list-item.
  5. Notice how the list item's html attributes are not updated in the DOM. They are updated in the table elements inside the shadow root, but not at the list-item attribute level.

Reproduction Version

2.12.0

Relevant Info

Independent of browser or operating system

Regression?

N/A

Priority impact

impact - p2 - want for an upcoming milestone

Impact

Impacts the Map Viewer team when users rename layer's and tables in filtered lists.

Calcite package

  • @esri/calcite-components
  • @esri/calcite-components-angular
  • @esri/calcite-components-react
  • @esri/calcite-design-tokens
  • @esri/eslint-plugin-calcite-components

Esri team

ArcGIS Map Viewer

@sagewall sagewall added 0 - new New issues that need assignment. bug Bug reports for broken functionality. Issues should include a reproduction of the bug. needs triage Planning workflow - pending design/dev review. labels Sep 3, 2024
@github-actions github-actions bot added ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. calcite-components Issues specific to the @esri/calcite-components package. impact - p2 - want for an upcoming milestone User set priority impact status of p2 - want for an upcoming milestone labels Sep 3, 2024
@driskull
Copy link
Member

driskull commented Sep 3, 2024

I think for this use case we can maybe introduce a filter method that a user can call when they need to do so. What do you think @jcfranco?

@geospatialem geospatialem added enhancement Issues tied to a new feature or request. p - medium Issue is non core or affecting less that 60% of people using the library estimate - 3 A day or two of work, likely requires updates to tests. needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. has workaround Issues have a workaround available in the meantime. and removed bug Bug reports for broken functionality. Issues should include a reproduction of the bug. needs triage Planning workflow - pending design/dev review. labels Sep 3, 2024
@geospatialem
Copy link
Member

As a workaround the above can currently be achieved by removing and re-creating the element.

@sagewall
Copy link
Author

sagewall commented Sep 3, 2024

Sorry, I was just looking at this again and it looks like the property changes I'm making to the list item in the JavaScript code are not reflected in the DOM. Only the table elements inside the shadow DOM seem to be updated. The calcite list item element shows the old property values. I was looking at the wrong element when I wrote the issue. I'll update the description up top.
image

@driskull
Copy link
Member

driskull commented Sep 3, 2024

They are being updated, they are just not reflected to attributes. The properties are updated.

@geospatialem geospatialem added this to the 2025-03-25 - Mar Release milestone Sep 6, 2024
@geospatialem geospatialem removed the needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. label Sep 6, 2024
@geospatialem geospatialem added the needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. label Feb 24, 2025
@geospatialem geospatialem removed this from the 2025-03-25 - 3.1.0 Mar Release milestone Feb 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 - new New issues that need assignment. ArcGIS Map Viewer Issues logged by ArcGIS Map Viewer team members. calcite-components Issues specific to the @esri/calcite-components package. enhancement Issues tied to a new feature or request. estimate - 3 A day or two of work, likely requires updates to tests. has workaround Issues have a workaround available in the meantime. impact - p2 - want for an upcoming milestone User set priority impact status of p2 - want for an upcoming milestone needs milestone Planning workflow - pending milestone assignment, has priority and/or estimate. p - medium Issue is non core or affecting less that 60% of people using the library
Projects
None yet
Development

No branches or pull requests

3 participants