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

fix: URL having Anchor element isn't displaying as expected - EXO-73493 - Meeds-io/meeds#2665. #4323

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

akhanfir
Copy link
Contributor

@akhanfir akhanfir commented Jan 7, 2025

Before this change, on Activity stream, click on news URL having anchors, news displayed having comment drawer opened. To resolve this problem, add a condition before opening the comments drawer if the hash is not an id of the comment add the id of the hash in the URL without opening it from the drawer. After this change, the news is displayed focusing on anchor without opening the comments.

…mme prévu - EXO-73493 - Meeds-io/meeds#2665.

Avant ce changement, sur le flux d'activité, cliquez sur l'URL de news contenant des ancres, les news affichées ayant le tiroir de commentaires ouvert. Pour résoudre ce problème, ajoutez une condition avant l'ouverture du tiroir de commentaires si le hash n'est pas un id du commentaire ajouter l'id du hash dans l'URL sans l'ouvrir du tiroir. Après ce changement, les news s'affichent en se concentrant sur l'ancre sans ouvrir les commentaires.
@akhanfir akhanfir enabled auto-merge January 7, 2025 07:50
@akhanfir akhanfir requested a review from mkrout January 7, 2025 07:50
@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Jan 7, 2025
Copy link

sonarqubecloud bot commented Jan 7, 2025

@akhanfir akhanfir changed the title correction : l'URL contenant un élément d'ancrage ne s'affiche pas comme prévu - EXO-73493 - Meeds-io/meeds#2665. fix: URL having Anchor element isn't displaying as expected - EXO-73493 - Meeds-io/meeds#2665. Jan 7, 2025
@akhanfir akhanfir merged commit 0c2a6b2 into feature/maintenance Jan 8, 2025
7 checks passed
@akhanfir akhanfir deleted the TASK-73493 branch January 8, 2025 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
partialCIBuild Perform Partial CI Build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants