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

Add LinkedObjectsBox component to mobile detail views #647

Merged
merged 4 commits into from
Jul 26, 2024

Conversation

leewyatt
Copy link
Collaborator

@leewyatt leewyatt commented Jul 25, 2024

  1. Introduced a new LinkedObjectsBox component to display related objects uniformly across various mobile detail pages. Updated existing pages such as MobileCompanyDetailsPage and MobileBookDetailsPage to incorporate this new component. This change will enhance consistency and maintainability in showing related objects.
  2. Refactor WelcomePage components to IntroPane and IntroCard

Introduced a new `LinkedObjectsBox` component to display related objects uniformly across various mobile detail pages. Updated existing pages such as `MobileCompanyDetailsPage` and `MobileBookDetailsPage` to incorporate this new component. This change will enhance consistency and maintainability in showing related objects.
Updated the mobile.css and Java files to reflect the renaming of WelcomePage to IntroPane and IntroCard. Simplified and adjusted styling for consistency and added an image credits file.
Replaced the static array of PageDate objects with a List to enable shuffling of the intro pages. This ensures a different order of pages each time the application runs, enhancing user experience.
Renamed the PageDate record to IntroCardData for better clarity and updated the respective variable references throughout the IntroPane class. This enhances code readability and aligns with naming conventions focused on introducing cards.
@dlemmermann dlemmermann merged commit 53b6679 into develop Jul 26, 2024
2 checks passed
@dlemmermann dlemmermann deleted the fetaure-linked-objects branch September 5, 2024 14:33
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