Skip to content

[Bug] Missing toolbar in EditVariationAttributesFragment in tablet layout #14097

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

Open
wants to merge 2 commits into
base: trunk
Choose a base branch
from

Conversation

samiuelson
Copy link
Contributor

Add toolbar to EditVariationAttributesFragment

WOOMOB-481

Description

This PR adds a toolbar to EditVariationAttributesFragment that was missing in the dual-pane layout (e.g. on tablet).

Steps to reproduce

On tablet device:

  1. Go to Products and select variable product
  2. Click Variations, and select a variation
  3. Click Attributes
  4. Notice the toolbar is missing

Testing information

Verify that the toolbar is shown both in dual pane and single pane layout.

The tests that have been performed

Above.

Images/gif

image

  • I have considered if this change warrants release notes and have added them to RELEASE-NOTES.txt if necessary. Use the "[Internal]" label for non-user-facing changes.

@samiuelson samiuelson marked this pull request as ready for review May 20, 2025 16:00
@samiuelson samiuelson added the feature: variation details Related to adding or editing variations and attributes. label May 20, 2025
@samiuelson samiuelson added this to the 22.5 milestone May 20, 2025
@samiuelson samiuelson added type: bug A confirmed bug. category: tablet Specific to tablet devices such as a Galaxy Tab or an iPad. labels May 20, 2025
@samiuelson samiuelson changed the title Add toolbar to EditVariationAttributesFragment [Bug] Missing toolbar in EditVariationAttributesFragment in tablet layout May 20, 2025
@samiuelson samiuelson requested a review from kidinov May 20, 2025 16:01
@wpmobilebot
Copy link
Collaborator

📲 You can test the changes from this Pull Request in WooCommerce-Wear Android by scanning the QR code below to install the corresponding build.
App Name WooCommerce-Wear Android
Platform⌚️ Wear OS
FlavorJalapeno
Build TypeDebug
Commit7b3abc2
Direct Downloadwoocommerce-wear-prototype-build-pr14097-7b3abc2.apk

@kidinov kidinov self-assigned this May 20, 2025
@wpmobilebot
Copy link
Collaborator

📲 You can test the changes from this Pull Request in WooCommerce Android by scanning the QR code below to install the corresponding build.

App Name WooCommerce Android
Platform📱 Mobile
FlavorJalapeno
Build TypeDebug
Commit7b3abc2
Direct Downloadwoocommerce-prototype-build-pr14097-7b3abc2.apk

@@ -77,6 +82,15 @@ class EditVariationAttributesFragment :

override fun getFragmentTitle() = getString(R.string.product_attributes)

private fun setupToolbar() {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that other fragments use setupTabletSecondPaneToolbar extension. Should we do the same?

@kidinov kidinov self-requested a review May 21, 2025 07:30
Copy link
Contributor

@kidinov kidinov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I'd just suggest using the extension to maintain consistency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: tablet Specific to tablet devices such as a Galaxy Tab or an iPad. feature: variation details Related to adding or editing variations and attributes. type: bug A confirmed bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants