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

ALFMOB-161: Incorrect Wishlist Header when opened from bottom bar #40

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JoaoPinhoMinder
Copy link
Contributor

@JoaoPinhoMinder JoaoPinhoMinder commented Mar 5, 2025

Ticket

https://mindera.atlassian.net/browse/ALFMOB-161

Description

The Wishlist toolbar was being defined in WishlistView, causing it to always use the same header. regardless of how the view was opened.

How it was fixed:

  • Moved toolbar definition to ViewFactory, ensuring the correct header is applied based on whether the view is opened from the bottom bar or via a push event.

Evidences

@JoaoPinhoMinder JoaoPinhoMinder changed the title ALFMOB-60: Decouple Navigation from Presentation - Screen type ALFMOB-161: Incorrect Wishlist Header when opened from bottom bar Mar 5, 2025
@JoaoPinhoMinder JoaoPinhoMinder force-pushed the bug/ALFMOB-161-incorrect-wishlist-header-when-opened-from-bottom-bar branch from 02573cb to 3343293 Compare March 5, 2025 14:13
The **Wishlist toolbar** was being defined in `WishlistView`, causing it to always use the same header. regardless of how the view was opened.

**How it was fixed:**
- Moved toolbar definition to `ViewFactory`, ensuring the correct header is applied based on whether the view is opened from the **bottom bar** or via a **push event**.
@JoaoPinhoMinder JoaoPinhoMinder force-pushed the bug/ALFMOB-161-incorrect-wishlist-header-when-opened-from-bottom-bar branch from 3343293 to 14d5e51 Compare March 5, 2025 14:19
@JoaoPinhoMinder JoaoPinhoMinder marked this pull request as ready for review March 5, 2025 14:21
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