Closed Bug 1670907 Opened 4 years ago Closed 4 years ago

Customize Print button not responsive after being set on Toolbar

Categories

(Firefox :: Menus, defect, P1)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
83 Branch
Tracking Status
firefox-esr78 --- unaffected
firefox81 --- unaffected
firefox82 --- wontfix
firefox83 --- verified
firefox84 --- verified

People

(Reporter: vlucaci, Assigned: Gijs)

References

(Blocks 1 open bug, Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Affected versions

  • 83.0a1(2020-10-12)
  • 82.0a(20201012131351)

Affected platforms

  • macOS 10.14
  • Ubuntu 18.04
  • Windows 10x64
  • Windows 7x64

Steps to reproduce

  1. Launch FF.
  2. Go to the burger menu
  3. Go to Customize.
  4. Put the Print button anywhere on the Toolbar.

Expected result

  • Button is active and can be interacted with as soon as it has been set on the toolbar.

Actual result

  • Button is greyed out and cannot be interacted with.

Suggested Severity

  • Seeing as how this issue occurs on all devices used in our testing, but not blocking our access to the feature(there are multiple other ways to trigger modal) , I would S3 this issue

Regression range

Additional notes

Regressed by: 1663173

Does this only affect print? What about other buttons that aren't in the toolbar by default (e.g. "save page as...", "file open", etc.)?

Flags: needinfo?(vlad.lucaci)

Yes, I was only able to reproduce this with the Print button.

I just tried with all the other buttons and all are working accordingly and do not display this behavior.

https://drive.google.com/file/d/1Ht4EybZ84YGDrWjrutvAabPT96SfPdy9/view?usp=sharing

As you can see in this above attachment, only the Print button is greyed out.

Flags: needinfo?(vlad.lucaci)

When we move items in customize mode, we unwrap them, then move them, then wrap
them again. The unwrapping sets the command attribute back on the button, and
that also sets the disabled attribute on the button. The wrapping removes the
command attribute but leaves the disabled attribute.
When the disabled attribute is removed from the command, this does not
propagate to the button because the command attribute has not been put back.

To fix this, the patch avoids adding and immediately removing the command
attribute when moving items while in customize mode.

Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED

Considering this only affects print and a new window or restart works, this doesn't seem super high severity.

Severity: -- → S3
Priority: -- → P1
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/bd5fbe4e35a3
moving buttons in customize mode should not disable them, r=mconley

wontfix for 82 based on comment 4.

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
Has Regression Range: --- → yes
Has STR: --- → yes
Flags: qe-verify+

Hello,

I have verified this issue on 84.0a1(20201026092834) and 83.0b4(20201025174155) with Windows 10x64 , macOS 10.15.7 and Ubuntu 18.04x64.

This issue is only partially fixed.

Moving the print icon to the toolbar as in the STR above, will now lack functionality until the browser is restarted.
Would you prefer to close this one and open a new issue for the new behavior?

Flags: needinfo?(gijskruitbosch+bugs)
Blocks: 1673388

(In reply to Vlad Lucaci, QA (:vlucaci) from comment #8)

Hello,

I have verified this issue on 84.0a1(20201026092834) and 83.0b4(20201025174155) with Windows 10x64 , macOS 10.15.7 and Ubuntu 18.04x64.

This issue is only partially fixed.

Moving the print icon to the toolbar as in the STR above, will now lack functionality until the browser is restarted.
Would you prefer to close this one and open a new issue for the new behavior?

Ugh. Thanks for flagging this up. I filed bug 1673388 so we can track release status flags.

Flags: needinfo?(gijskruitbosch+bugs)

Closing this issue as per the comment above.

Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: