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

[Admin] Add anchors to repositories page #5427

Merged
merged 2 commits into from
Feb 19, 2025

Conversation

rldhont
Copy link
Collaborator

@rldhont rldhont commented Feb 18, 2025

To easily come back to the right position in the repositories page, anchors has been added.

@rldhont
Copy link
Collaborator Author

rldhont commented Feb 18, 2025

Tests have to be addded

@github-actions github-actions bot added this to the 3.10.0 milestone Feb 18, 2025
@rldhont rldhont force-pushed the admin-repositories-anchors branch from 37fecb8 to 7832151 Compare February 18, 2025 14:08
@rldhont rldhont marked this pull request as ready for review February 18, 2025 14:08
@rldhont rldhont added the run end2end If the PR must run end2end tests or not label Feb 18, 2025
@rldhont
Copy link
Collaborator Author

rldhont commented Feb 18, 2025

e2e tests added

Copy link
Member

@Gustry Gustry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lizmap/www/assets/css/admin.css Outdated Show resolved Hide resolved
lizmap/www/assets/css/admin.css Outdated Show resolved Hide resolved
@rldhont rldhont force-pushed the admin-repositories-anchors branch from 7832151 to 15a6cee Compare February 18, 2025 15:56
@rldhont
Copy link
Collaborator Author

rldhont commented Feb 18, 2025

@Gustry thanks for review

@Gustry
Copy link
Member

Gustry commented Feb 18, 2025

Side note, I will try to add the same behavior as on many websites, with visible anchor link, like https://github.com/3liz/lizmap-web-client/?tab=readme-ov-file

  1. Invisible
  2. Hover the title, it displays the anchor link

To easily come back to the right position in the repositories page, anchors has been added.
@rldhont rldhont force-pushed the admin-repositories-anchors branch from 15a6cee to b3a1371 Compare February 18, 2025 16:25
Copy link
Member

@Gustry Gustry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Sorry indeed, it's only in the admin part

@rldhont rldhont merged commit 17e1aa2 into 3liz:master Feb 19, 2025
18 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants