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] Allow current page to be excluded #2699

Open
ky940819 opened this issue Mar 20, 2024 · 1 comment · May be fixed by #2701
Open

[List] Allow current page to be excluded #2699

ky940819 opened this issue Mar 20, 2024 · 1 comment · May be fixed by #2701

Comments

@ky940819
Copy link
Contributor

Feature Request

Is your feature request related to a problem? Please describe.
There is currently no way to exclude the current requested page from the results of a List component.

This would be a beneficial feature in situations where a list is reused across multiple pages (e.g. via an Experience Fragment) but it is not desirable to include the current page in the list. For example, a "More In This Section" list which shows sibling pages that are not the current page, and adapts automatically based on what page is being viewed.

Currently, to achieve this an author would have to place a list on each page individually, and use the static list type and select all pages that are not the current page, and then maintain every list individually as pages are added/removed in the future.

Describe the solution you'd like
A simple checkbox in the List component dialog that, when checked, excludes the current request page from the list results.
This should work regardless of the list type, and should not effect the maximum number of items to be displayed.

Are there alternatives?
N/A

Documentation
N/A

ky940819 added a commit to ky940819/aem-core-wcm-components that referenced this issue Mar 20, 2024
Adds option to the V4 List component to exclude the current
page from the results.

----
fixes adobe#2699
@ky940819 ky940819 linked a pull request Mar 20, 2024 that will close this issue
ky940819 added a commit to ky940819/aem-core-wcm-components that referenced this issue Mar 20, 2024
Adds option to the V4 List component to exclude the current
page from the results.

----
fixes adobe#2699
ky940819 added a commit to ky940819/aem-core-wcm-components that referenced this issue Mar 20, 2024
Adds option to the V4 List component to exclude the current
page from the results.

----
fixes adobe#2699
@juw177
Copy link

juw177 commented Sep 23, 2024

Can this change be included? There are very few cases where List would need to include the current page.

ky940819 added a commit to ky940819/aem-core-wcm-components that referenced this issue Oct 8, 2024
Adds option to the V4 List component to exclude the current
page from the results.

----
fixes adobe#2699
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 a pull request may close this issue.

2 participants