Skip to content

fix: improve UX and clarity for Bulk Search#11358

Merged
cdrini merged 14 commits intointernetarchive:masterfrom
akramcodez:improvements-in-bulk-search-ui
Dec 11, 2025
Merged

fix: improve UX and clarity for Bulk Search#11358
cdrini merged 14 commits intointernetarchive:masterfrom
akramcodez:improvements-in-bulk-search-ui

Conversation

@akramcodez
Copy link
Copy Markdown
Contributor

Closes #11347

Changes

  • Implemented auto-scroll for Bulk Search steps: when a step card becomes enabled, it automatically scrolls into view.
  • Added activeStep visual indicator with a blue border/shadow to highlight the current step.
  • Updated carousel to clearly show that only the first work will be saved: added subtle transparency to other items.
  • Clarified spreadsheet instructions to explicitly indicate copying/pasting the table directly.
  • Minor UI tweaks for better visual clarity and user guidance.

Screen recording

video.mov

Copilot AI review requested due to automatic review settings October 16, 2025 07:44
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the user experience of the Bulk Search feature by adding visual feedback and clearer instructions. The changes focus on guiding users through a three-step process with better visual indicators and explicit instructions.

  • Implements auto-scrolling to active steps and adds visual highlighting with blue borders
  • Distinguishes the primary book result (first card) from secondary results using opacity and border styling
  • Adds instructional text to clarify the spreadsheet copy/paste workflow

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
openlibrary/components/BulkSearch/components/BulkSearchControls.vue Adds auto-scroll behavior, activeStep tracking, visual step highlighting, and instructional text for the bulk search workflow
openlibrary/components/BulkSearch/components/BookCard.vue Implements primary/secondary card styling with opacity differences and border highlighting
openlibrary/components/BulkSearch/components/TableRow.vue Passes isPrimary prop to BookCard to identify the first result
openlibrary/components/BulkSearch/components/MatchRow.vue Passes isPrimary prop to BookCard to identify the first result

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions github-actions bot added the Needs: Response Issues which require feedback from lead label Oct 16, 2025
@akramcodez
Copy link
Copy Markdown
Contributor Author

PTAL @cdrini, thanks.

Copy link
Copy Markdown
Collaborator

@cdrini cdrini left a comment

Choose a reason for hiding this comment

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

Nice great work @akramcodez ! A few small tweaks for styling 👍

@cdrini cdrini added Needs: Submitter Input Waiting on input from the creator of the issue/pr [managed] and removed Needs: Response Issues which require feedback from lead labels Nov 10, 2025
@github-actions github-actions bot removed the Needs: Submitter Input Waiting on input from the creator of the issue/pr [managed] label Nov 11, 2025
@akramcodez
Copy link
Copy Markdown
Contributor Author

@cdrini thanks for the suggestions, all implemented.

@github-actions github-actions bot added the Needs: Response Issues which require feedback from lead label Nov 11, 2025
@akramcodez
Copy link
Copy Markdown
Contributor Author

@cdrini if there's any problem, please let me know.

@cdrini
Copy link
Copy Markdown
Collaborator

cdrini commented Dec 2, 2025

Hi @akramcodez , please make sure to resolve all the cr comments (see https://github.com/internetarchive/openlibrary/blob/master/CONTRIBUTING.md#submitting-pull-requests ). That's what signals to me the PR is ready for another look!

Also note it seems like we lost the scrollIntoView logic with the latest changes. That will need to be restored.

@cdrini cdrini added Needs: Submitter Input Waiting on input from the creator of the issue/pr [managed] and removed Needs: Response Issues which require feedback from lead labels Dec 2, 2025
@github-actions github-actions bot removed the Needs: Submitter Input Waiting on input from the creator of the issue/pr [managed] label Dec 2, 2025
@akramcodez
Copy link
Copy Markdown
Contributor Author

Hi @cdrini, sorry about that I had pushed the changes but forgot to resolve the review comments. I've now resolved them and also restored the scrollIntoView logic.

@github-actions github-actions bot added the Needs: Response Issues which require feedback from lead label Dec 3, 2025
Copy link
Copy Markdown
Collaborator

@cdrini cdrini left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you @akramcodez

@cdrini cdrini merged commit f6c509a into internetarchive:master Dec 11, 2025
4 checks passed
mystic-06 pushed a commit to mystic-06/openlibrary that referenced this pull request Dec 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Response Issues which require feedback from lead

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improvements / Tweaks to Bulk Search UI

3 participants