-
Notifications
You must be signed in to change notification settings - Fork 3
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
Updating Apartment Page UI/Functionality V3 #309
Conversation
[diff-counting] Significant lines: 146. |
Visit the preview URL for this PR (updated for commit 7d0eb92): https://cu-apts-staging--pr309-updating-apartment-r-ni160rpr.web.app (expires Mon, 20 Nov 2023 23:51:54 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 096ac87b789b31770a01964fe0aaa92d563b9353 |
This PR looks great and implements a needed feature that provides users with more information about each apartment's reviews! Perhaps you could delete the handleShowMore function instead of commenting it out? Good job! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work Cyrus! This code is very needed and makes the user experience a lot better. I also really liked the comments that you put throughout. Looking forward to seeing it in action!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new segment doesn't work on admin page due to some problem while rendering AdminReviewComponent --> LabeledLinearProgress
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job on adapting new review bar design! This PR makes the score much easier to estimate visually with separated segments. Thanks for also fixing bug in admin page which previous does not render the new review bar properly.
* migrating changes * specification * fixed precentage of segment * adjust spacing * fixed bug * fixed bug * fixed sizing * size of landlord info --------- Co-authored-by: Thuy Pham <[email protected]> Co-authored-by: thuypham03 <[email protected]>
* migrating changes * specification * fixed precentage of segment * adjust spacing * fixed bug * fixed bug * fixed sizing * size of landlord info --------- Co-authored-by: Thuy Pham <[email protected]> Co-authored-by: thuypham03 <[email protected]>
Summary
This PR is to update the current User Interface of the apartment page. Instead of the linear scale for the feature ratings, there is now 5 subsections for each review scale.
Test Plan
Tested on resizing the browser window.
The Current Look:
Figma: