This is roughly ok, though I did find a few problems:
- "Unmute" links keeps pointing to "+subscribe" instead of "+mute" (+mute page was changed to support both muting/unmuting, and doesn't redirect to +subscribe when bug mail is muted anymore); this means that people will be sent to the wrong page if they have no JS
- When there is a mute on a bug, but no subscription, clicking on "Subscribe" fails to pop-up the window; also, going to +subscribe page renders an empty form which doesn't work in that case (this two are likely related)
- When one tries to "Subscribe" originally (or unmute when they have an old subscription, or unsubscribe yourself), subscription overlay pop-up stays loaded when you click on check-mark to apply the action even though the action succeeds; this is especially bad if there's an error since then the error pop-up is below the subscription overlay
Some of this might be due to the split of the subscription actions and subscribers list into separate "portlets".
This is roughly ok, though I did find a few problems:
- "Unmute" links keeps pointing to "+subscribe" instead of "+mute" (+mute page was changed to support both muting/unmuting, and doesn't redirect to +subscribe when bug mail is muted anymore); this means that people will be sent to the wrong page if they have no JS
- When there is a mute on a bug, but no subscription, clicking on "Subscribe" fails to pop-up the window; also, going to +subscribe page renders an empty form which doesn't work in that case (this two are likely related)
- When one tries to "Subscribe" originally (or unmute when they have an old subscription, or unsubscribe yourself), subscription overlay pop-up stays loaded when you click on check-mark to apply the action even though the action succeeds; this is especially bad if there's an error since then the error pop-up is below the subscription overlay
Some of this might be due to the split of the subscription actions and subscribers list into separate "portlets".