Skip to content

OTWO-7681 Improvement functionlity of openhub for new UI/UX#1917

Open
bd-vaibhav wants to merge 2 commits into
OTWO-7546from
OTWO-7681
Open

OTWO-7681 Improvement functionlity of openhub for new UI/UX#1917
bd-vaibhav wants to merge 2 commits into
OTWO-7546from
OTWO-7681

Conversation

@bd-vaibhav

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

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 updates Open Hub’s UI/UX by introducing a global “page loader” overlay for slow navigations, refining search results pagination placement/styling, and adjusting password reset/account settings UX for better security and correctness.

Changes:

  • Added a global page loader partial + styling + JS behavior to display a spinner overlay when navigating to selected “slow” pages.
  • Moved and restyled search pagination info so it appears between the filter bar and results.
  • Improved password reset messaging (anti-enumeration) and gated the “change password” settings card to only appear for the signed-in user.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
config/locales/en.yml Adds i18n string for loader and updates password reset success copy.
app/views/shared/_search_dingus.html.haml Repositions pagination info render relative to the filter bar/results.
app/views/shared/_page_loader.html.haml Adds the page loader overlay markup (spinner + loading text).
app/views/passwords/create.html.haml Updates success-state heading copy.
app/views/layouts/application.html.haml Renders the page loader globally on all pages.
app/views/accounts/settings.html.haml Shows “Change password” only when viewing your own account settings.
app/assets/stylesheets/search-dingus.sass Moves pagination info styling to a global rule and adjusts presentation.
app/assets/stylesheets/page_loader.sass Introduces overlay/spinner styling for the global loader.
app/assets/javascripts/page_loader.js Adds click-based detection to show/hide the loader around slow navigations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread app/assets/javascripts/page_loader.js
Comment thread app/assets/javascripts/page_loader.js
Comment thread app/views/shared/_page_loader.html.haml
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.

2 participants