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

Minor UI Fixes #1679

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Minor UI Fixes #1679

wants to merge 2 commits into from

Conversation

sahitya-chandra
Copy link

@sahitya-chandra sahitya-chandra commented Dec 23, 2024

Resolves #1678

UI Fixes

  • Appbar aligned with topbar (Img 1)
  • Stickey behavior of BreadCrumbComponent fixed (Video)
  • Filter button removed from (Img 2)
  • Filter button is aligned to flex end (Img 3)

Img 1
Screenshot from 2024-12-23 19-56-32

Img 2
Screenshot from 2024-12-23 19-55-54

Img 3
Screenshot from 2024-12-23 20-32-43

Video
Screencast from 2024-12-23 19-56-50.webm

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

@sahitya-chandra sahitya-chandra changed the title Minnor UI Fixes Minor UI Fixes Dec 23, 2024
@rajveeerr
Copy link
Contributor

rajveeerr commented Dec 24, 2024

Some of these issues have already been resolved but are not yet visible in production because the main and production branches have not been synced. This will be addressed soon, as confirmed by @devsargam and @siinghd:

  1. Sticky Behavior of the BreadCrumbComponent:
    Resolved in bug: unnecessary space between header and breadcrumb #1627, fixing the gap between breadcrums and navbar.

  2. Filter Button Alignment:
    Addressed in fix: iframe-video positioning and width fixes along with some accordion styling fixes  #1625, ensuring the filter button is properly aligned to the flex-end.

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.

bug: UI Improvment
2 participants