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

Simplify MobileCategoryHeader instantiation #629

Merged
merged 2 commits into from
Jul 13, 2024

Conversation

leewyatt
Copy link
Collaborator

@leewyatt leewyatt commented Jul 10, 2024

  1. Remove header image in detail pages
  2. Removed redundant overriden method goBackLink() from MobileCategoryHeader in all details pages. This simplifies how MobileCategoryHeader is instantiated, reducing code duplication and making the code more maintainable.

Removed redundant overriden method `goBackLink()` from `MobileCategoryHeader` in all details pages. This simplifies how `MobileCategoryHeader` is instantiated, reducing code duplication and making the code more maintainable.
@leewyatt leewyatt marked this pull request as draft July 10, 2024 12:59
@leewyatt leewyatt marked this pull request as ready for review July 10, 2024 13:09
The image bindings that previously set header images for various detailed pages have been removed. As the specific line of code setting these bindings was removed in each of the detail pages, the headers for these pages will no longer display preview images based on their respective objects (e.g., tutorial, person, video, tip, book, blog, app, tool, library).
@dlemmermann dlemmermann merged commit 43223b5 into develop Jul 13, 2024
2 checks passed
@dlemmermann dlemmermann deleted the enhancement-remove-go-backlink 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