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

Comment Section in the shifting detail page is stretching the page #6637

Open
nihal467 opened this issue Nov 15, 2023 · 17 comments
Open

Comment Section in the shifting detail page is stretching the page #6637

nihal467 opened this issue Nov 15, 2023 · 17 comments
Assignees

Comments

@nihal467
Copy link
Member

nihal467 commented Nov 15, 2023

Describe the bug

When we post multiple comment on a shifting detail page, it keep on stretching the page to the bottom

To Reproduce
Steps to reproduce the behavior:

  1. Go to any existing shifting detail page and post multiple comments ( https://care.coronasafe.in/shifting/e785d651-9fde-4279-ac33-e9282296dc10 )
  2. See error

Expected behavior

It shouldn't be stretching the page a lot and the entire page should be effectively utilizing the white space

Screenshots

image

@parthksingh1
Copy link

@rithviknishad Please assign me this issue .To resolve the bug, I will inspect and adjust the CSS styles associated with the comment section on shifting detail pages, ensuring responsive design and dynamic sizing. Will Thoroughly test the changes with different comment quantities and devices to ensure optimal page layout

@konavivekramakrishna
Copy link
Contributor

@rithviknishad

A simple solution would be to passing perPage 5 or 6. by default per Page is 14, causing the entire page to stretch significantly

image

@Bhavik-ag
Copy link
Contributor

@rithviknishad @nihal467 Here are two simple possible solutions

  1. We can shift the patient details section to the right side of grid and setting comment perPage limit to 5.

image

  1. We can make patient details page expand full width and comments and rest section side by side.

image

@rishbanerjee
Copy link

Hey @rithviknishad @nihal467 I tried to see this error on the webpage but when I try to post a comment it shows an error "Not found" , is this issue still relevant?

@senthil-athiban
Copy link

Hey @nihal467. We can enhance the user experience by adding a view more button. Based on that, when the user clicks on it, an extra 5 more comments will be visible, and we'll repeat the same until the comment exists. And intially Top 5 comments will be only displayed.

@Thanush19
Copy link

Hey, if the issue is not completed, I can work on this.

@Thanush19
Copy link

Shifting.details._.CARE.mp4

i have submitted my PR #7364, kindly review it , if any changes need to be done , intimate me @rithviknishad

@Thanush19
Copy link

@rithviknishad , is there any corrections needed to be done ? if so, intimate me...?

@Thanush19
Copy link

@rithviknishad , whats the status on this PR?? is any changes required...

@rithviknishad
Copy link
Member

Hey, not sure if you've missed it, but I've requested changes in your PR two days ago

@Thanush19
Copy link

Thanush19 commented Mar 12, 2024

@rithviknishad You instructed me to reuse the existing type, but it lacks the required types for comments. Additionally, you mentioned reusing the ReadMore.tsx component, but when I attempt to do so, it doesn't align with the Comment Box. Modifying the CSS of the ReadMore component might disrupt the UI of other components, so I opted to write new code to address this issue.

@rithviknishad
Copy link
Member

Alright. ReadMore component is fine.

But any type is not allowed when we already have types for it.

If you look carefully, proper types are already present for created_by_object in the same file too. You just have to re-use it.

image

@Thanush19
Copy link

@rithviknishad , i've submitted new PR ,correcting all the changes u asked for , and also added Multilingual support for all the Button i have used , check my PR #7402

@Thanush19
Copy link

hey @rithviknishad , whats the status on my PR....

@nihal467
Copy link
Member Author

@Thanush19 Unassigning you from this issue due to inactivity. Feel free to reassign yourself if you are still working on it.

@NikhilGolla72
Copy link

@rithviknishad hello,is this issue still open? if yes, I would like to work on this issue.

@Jacobjeevan Jacobjeevan self-assigned this Oct 18, 2024
@Jacobjeevan Jacobjeevan moved this from Up Next to In Progress in Care Oct 21, 2024
@Jacobjeevan Jacobjeevan moved this from In Progress to Triage in Care Oct 24, 2024
@Jacobjeevan
Copy link
Contributor

Holding this on the back burner now, as shifting details page might get redesigned (as grid/table format). If so, could redesign this page along with that 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment