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

Refactor MobilePersonDetailsPage and add MobilePersonDetailView #633

Merged
merged 5 commits into from
Jul 22, 2024

Conversation

leewyatt
Copy link
Collaborator

Simplified MobilePersonDetailsPage by removing a complex header and details content creation. Introduced MobilePersonDetailView supporting the view with person details. The changes include the display of badges indicating the person's contribution status and an update in styling the details page.

Simplified MobilePersonDetailsPage by removing a complex header and details content creation. Introduced MobilePersonDetailView supporting the view with person details. The changes include the display of badges indicating the person's contribution status and an update in styling the details page.
@leewyatt leewyatt marked this pull request as ready for review July 19, 2024 16:15
leewyatt and others added 4 commits July 20, 2024 01:16
The home page and person detail view of the mobile application have been refactored. Unused imports were removed, the logic to create preview views was moved to the CategoryPreviewView class, making the code more organized. Moreover, the avatar and linked objects in the person detail view were updated to enhance the user experience. Finally, some minor cosmetic changes were made to the linked badges and CSS styles.
The updates rearrange the order of operations for resetting the avatar, text, and boxes when a person's detail view is refreshed. Additionally, the updateBadge() and updatePersonLinkedObjectBox() functions have been adjusted to directly receive the person object as a parameter and the labels of badges have been corrected.
Added a new category for videos in the mobile home page.
@dlemmermann dlemmermann merged commit 3b6ef06 into develop Jul 22, 2024
2 checks passed
@dlemmermann dlemmermann deleted the feature-mobile-person-details-view branch September 5, 2024 14:32
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