Skip to content

Conversation

@adriendupuis
Copy link
Contributor

@adriendupuis adriendupuis commented Mar 6, 2025

Question Answer
JIRA Ticket IBX-4064
Versions All
Edition All
  • Update ez-icons.svg from dev doc one. Some search nav icons were missing.
    • Fix first page icon so it's now visible
    • I didn't add the new icons to docs/persona_paths/icons.md as they don't adapt well, (for example, they don't accept the color). If needed, it could be asked to design team to fix them.
    • No icon from previous version changed:
  • Hide last page icon as meaning less (nothing interesting there, this is not even the last page in some case)
  • Prevent last clicked first/previous/next/last page button from being highlighted.
  • docs/css/instantsearch.jswas introduced in IBX-4064: Search page #322 and duplicates docs/js/instantsearch.js; It was probably a paste error while copying dev doc's one.

Before
image

After
image

Checklist

  • Text renders correctly
  • Text has been checked with vale
  • Description metadata is up to date
  • Redirects cover removed/moved pages
  • Added link to this PR in relevant JIRA ticket or code PR

probably a copy-paste error when adapting dev doc's one.
- Fix first page link icon
- hide "last page" link (this is not even the last page)
- Fix first page link icon
- hide "last page" link (this is not even the last page)
@github-actions
Copy link

github-actions bot commented Mar 6, 2025

Preview of modified Markdown: no Markdown change to preview.

@adriendupuis adriendupuis mentioned this pull request Mar 6, 2025
5 tasks
@adriendupuis adriendupuis changed the title Ibx 4064 enh search page IBX-4064: Fix search page nav Mar 6, 2025
@adriendupuis adriendupuis requested a review from dabrt March 6, 2025 15:10
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 6, 2025

@adriendupuis adriendupuis merged commit 22cdddd into master Mar 13, 2025
7 checks passed
@adriendupuis adriendupuis deleted the IBX-4064-enh-search-page branch March 13, 2025 13:52
adriendupuis added a commit that referenced this pull request Mar 13, 2025
* rm docs/css/instantsearch.js
* instantsearch.js: Fix navigation
    - Fix first page link icon
    - hide "last page" link (this is sometimes not even the last page)
* instantsearch.css: don't highlight last clicked prev/next button

(cherry picked from commit 22cdddd)
adriendupuis added a commit that referenced this pull request Mar 13, 2025
* rm docs/css/instantsearch.js
* instantsearch.js: Fix navigation
    - Fix first page link icon
    - hide "last page" link (this is sometimes not even the last page)
* instantsearch.css: don't highlight last clicked prev/next button

(cherry picked from commit 22cdddd)
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