Skip to content

Tanvi-Dashboard UI Fixes & Weekly Summary Enhancements#3372

Merged
one-community merged 8 commits into
developmentfrom
clean-fix-feature
Apr 7, 2025
Merged

Tanvi-Dashboard UI Fixes & Weekly Summary Enhancements#3372
one-community merged 8 commits into
developmentfrom
clean-fix-feature

Conversation

@TanviAnantula
Copy link
Copy Markdown
Contributor

@TanviAnantula TanviAnantula commented Apr 5, 2025

Description

Responsive UI fixes for Dashboard components on small screens (<768px), including Weekly Summary, Team Member Tasks, and Badges sections. Also ensures scrollable task table and consistent layout.

Related PRS (if any):

This is frontend PR
This PR replaces the earlier branch vijay-anand-dashboard-ui-fixes#2684 with a clean and conflict-free implementation of the intended changes.
To test this frontend PR, you need to checkout the development branch

Main changes explained:

  • Fixed UI issue of the "Close this window" button in the Weekly Summarycomponent for screen sizes less than 768px
  • Fixed UI issue of the Hours buttons in Team Member Tasks tab (under Tasks and Timelogs) for screen sizes less than 768px
  • Fixed UI issue of the "Badge Report" button in the Badges tab (under Tasks and Timelogs) for screen sizes less than 768px
  • Fixed UI issue of the Task table not being scrollable in the Team Member Tasks tab for screen sizes less than 768px

How to test:

Checkout the branch: git checkout clean-fix-feature
Run npm install
npm start
Clear browser cache/site data
Log in as Admin or Owner user
Go to Dashboard → Submit Weekly Summary and check if "Close this window" button is responsive under 768px
Go to Dashboard → Tasks and Timelogs → Tasks and check if hours buttons become dropdowns under 768px
Go to Dashboard → Tasks and Timelogs → Badges and check if "Badge Report" button is properly aligned under 768px
Go to Dashboard → Tasks and Timelogs → Tasks and check if task table is scrollable under 768px
Verify all the above components also display correctly on desktop/large screen

Screenshots or videos of changes:

Screenshot 1

image

image

Updated Screenshot 2

image

Note:

All changes have been tested on both mobile and desktop screen sizes.

@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 5, 2025

Deploy Preview for highestgoodnetwork-dev ready!

Name Link
🔨 Latest commit be995b3
🔍 Latest deploy log https://app.netlify.com/sites/highestgoodnetwork-dev/deploys/67f35378519d250008529def
😎 Deploy Preview https://deploy-preview-3372--highestgoodnetwork-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

ppsahu
ppsahu previously approved these changes Apr 5, 2025
Copy link
Copy Markdown
Contributor

@ppsahu ppsahu left a comment

Choose a reason for hiding this comment

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

Hi, I have tested this PR in mobile view, and all the changes are working as expected and as per the screenshots. Below is the screen recording for reference.

Screen.Recording.2025-04-05.at.2.37.29.AM.mov

Copy link
Copy Markdown

@palankigreeshma1109 palankigreeshma1109 left a comment

Choose a reason for hiding this comment

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

I have reviewed this PR and followed all the steps mentioned and everything is working as expected.

  1. Go to Dashboard → Submit Weekly Summary and check if "Close this window" button is responsive under 768px - It's responsive.

Screenshot (83)

2.Go to Dashboard → Tasks and Timelogs → Tasks and check if hours buttons become dropdowns under 768px - Yes and task table is also scrollable.

Screenshot (84)
ee1184)

3.Go to Dashboard → Tasks and Timelogs → Badges and check if "Badge Report" button is properly aligned under 768px - It's properly alligned.

Screenshot (85)

Attached screenshots for your reference.

Copy link
Copy Markdown
Contributor

@pavanputti pavanputti left a comment

Choose a reason for hiding this comment

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

I checked out the clean-fix-feature branch, ran npm install, and started the application using npm start. After clearing the browser cache and site data, I logged in as an Admin user and navigated to the Dashboard.

Verified the "Close this window" button in the Weekly Summary modal
Screenshot 2025-04-05 at 3 09 04 PM

The button was responsive on screen widths under 768px.

Ensured the task table remained scrollable and the hours buttons became dropdowns in the Tasks section

Screenshot 2025-04-05 at 3 13 27 PM Screenshot 2025-04-05 at 3 11 54 PM

The task table remained scrollable, and the hours buttons were correctly displayed as dropdowns on smaller screens.

Verified the alignment of the "Badge Report" button in the Badges tab

Screenshot 2025-04-05 at 3 10 39 PM

The button was properly aligned on screen widths under 768px.
Everything works as expected.

pavanputti
pavanputti previously approved these changes Apr 5, 2025
rsripathi2906
rsripathi2906 previously approved these changes Apr 6, 2025
Copy link
Copy Markdown

@rsripathi2906 rsripathi2906 left a comment

Choose a reason for hiding this comment

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

Tested

  • Verified the "Close this window" button in Weekly Summary is responsive and functional under <768px.
  • Checked Hours buttons in Team Member Tasks — correctly render as dropdowns on small screens.
  • Badge Report button in Badges tab is well-aligned for mobile view.
  • Task table is now scrollable under 768px — works as expected.
  • Also confirmed that all components display correctly on larger screens.
  • Everything looks good to me — approving the PR.
Screen.Recording.2025-04-05.200650.mp4

@one-community
Copy link
Copy Markdown
Member

Thank you all, merging!

@one-community one-community merged commit 338e972 into development Apr 7, 2025
6 checks passed
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.

6 participants