#4191 closed defect (fixed)
Neighborhood view: Palette stays up after connecting with AP (mouse)
| Reported by: | erikos | Owned by: | erikos |
|---|---|---|---|
| Priority: | High | Milestone: | |
| Component: | Sugar | Version: | 0.97.x |
| Severity: | Major | Keywords: | regression, r+, olpc-testing-pending |
| Cc: | humitos, erikos | Distribution/OS: | Unspecified |
| Bug Status: | Assigned |
Description
Build: os11
Steps to reproduce:
- go to neighbourhood view
- click on AP icon with the mouse
- click on the 'connect' option
- the PW-dialogue comes up
- enter credentials
- the AP Palette stays up
Attachments (1)
Change History (7)
comment:1 Changed 11 years ago by erikos
- Bug Status changed from Unconfirmed to Assigned
- Keywords regression added; triage removed
- Milestone changed from Unspecified by Release Team to 0.98
- Priority changed from Unspecified by Maintainer to High
- Severity changed from Unspecified to Major
- Version changed from Unspecified to 0.97.x
comment:2 follow-up: ↓ 3 Changed 11 years ago by humitos
- Cc humitos added
I think there is missing a self.popup(immediate=True) in the connect / disconnect _activate_cb: sugar/src/jarabe/desktop/networkviews.py
comment:3 in reply to: ↑ 2 Changed 11 years ago by manuq
Replying to humitos:
I think there is missing a self.popup(immediate=True) in the connect / disconnect _activate_cb: sugar/src/jarabe/desktop/networkviews.py
Yes! an immediate popdown, I thought the same, as we did for Browse. The attached patch does it. However this affects every PaletteMenuItem with an 'active' state, so maybe a general solution is possible. See #4253 .
comment:4 Changed 11 years ago by manuq
- Cc erikos added
- Keywords r? added
comment:5 Changed 11 years ago by erikos
- Keywords r+ olpc-testing-pending added; r? removed
- Resolution set to fixed
- Status changed from new to closed
Fixed in f5fee8a1445f76d3b04497671a93179329fe1efb toolkit.
Note: See
TracTickets for help on using
tickets.
Should be fixed for 0.98.1 latest.