Skip to content

Conversation

@FaridaSabrin
Copy link

Closes: #1284

Describe the add-ons

  • Converted the entire Search module UI from custom components to Material-UI (MUI).
  • Updated layout using MUI Grid, Container, and responsive breakpoints for better mobile support.
  • Replaced old elements with MUI Paper, Tabs, Typography, Dialog, Alert, and Icons for a consistent design.
  • Added loading indicators, success and error alerts, and search result count for improved user feedback.
  • Enhanced spacing, alignment, and overall visual appearance.
  • Implemented keyboard shortcuts (Help, Refresh, Tab switch, etc.).
  • Shortcuts are partially working but facing browser-level conflicts and event handling issues.
  • Could not identify the exact cause or file for the issue, further debugging is needed.
  • All core features are tested and working fine after the MUI migration.

Screenshots

Screenshot 2025-10-23 115910

Note to reviewers

  • The UI migration task for the Search module is completed and verified.
  • Keyboard shortcut conflicts need additional debugging at the browser or event-handling level.
  • Please consider this PR as completion of the UI migration part of Issue Migrate Search module UI to MUI and ensure keyboard shortcuts #1284.
  • Thank you for assigning this task. It helped me gain more experience with MUI and responsive UI design.

@vercel
Copy link

vercel bot commented Oct 23, 2025

Someone is attempting to deploy a commit to the avdheshvarshney's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

@Avdhesh-Varshney
Copy link
Collaborator

@FaridaSabrin Update your PR by taking the latest pull.
Try to refactor the code something like home module. So, that all functions are segregate from the rendering component.
I've already setup the apis base to be clear. So, just directly use those functions directly.

@FaridaSabrin
Copy link
Author

@Avdhesh-Varshney Ok

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 this pull request may close these issues.

Migrate Search module UI to MUI and ensure keyboard shortcuts

2 participants