Closed Bug 1545011 Opened 1 year ago Closed 1 year ago

[remote-dbg-next] Make the connection error message on sidebar to be closable

Categories

(DevTools :: about:debugging, defect, P1)

defect

Tracking

(firefox68 verified)

VERIFIED FIXED
Firefox 68
Tracking Status
firefox68 --- verified

People

(Reporter: daisuke, Assigned: ogasidlo)

References

(Blocks 1 open bug)

Details

(Whiteboard: [remote-debugging-reserve])

Attachments

(1 file)

This is a followup of bug 1540609.

Including above bug, we show some connection error messages. However those messages can not close by user. In this bug, make the messages to be closable by user.

After https://phabricator.services.mozilla.com/D27645 lands, this should be a quick fix.
Please add isClosable: true, to all message components that need a closing button.

e.g.
here: https://searchfox.org/mozilla-central/source/devtools/client/aboutdebugging-new/src/components/sidebar/SidebarRuntimeItem.js#79

https://searchfox.org/mozilla-central/source/devtools/client/aboutdebugging-new/src/components/sidebar/SidebarRuntimeItem.js#104

Regarding this patch, we also need to update following function with the additional parameter:

renderMessage(flag, level, localizationId, className) {...} renderMessage(flag, isCloseable,level, localizationId, className) {...}

return Message(
  {
    level,
    key: className,
    className,
    isCloseable
  },
Assignee: nobody → ogasidlo
Status: NEW → ASSIGNED

Make the connection error + warning messages on the sidebar closeable

Priority: -- → P1
Whiteboard: [remote-debugging-reserve]
Pushed by ogasidlo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c2d1f25ac62f
added closing button to error messages on sidebar r=daisuke,jdescottes
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68

Verified as fixed on Firefox Nightly 68.0a1 (2019-05-16) on Windows 10 x 64, Mac OS X 10.14 and on Ubuntu 18.04 x64.

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