UI: Subscribe Dialog - Statusbar should be at bottom of dialog

VERIFIED FIXED in mozilla1.0

Status

P3
normal
VERIFIED FIXED
18 years ago
10 years ago

People

(Reporter: jglick, Assigned: andreww)

Tracking

Trunk
mozilla1.0

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(3 attachments, 14 obsolete attachments)

1.09 KB, patch
hwaara
: review+
Details | Diff | Splinter Review
8.02 KB, image/gif
Details
2.02 KB, patch
asa
: review+
Bienvenu
: superreview+
Details | Diff | Splinter Review
(Reporter)

Description

18 years ago
In the Subscribe Dialog, the statusbar should be at the bottom of the dialog,
(like Search dialog) and not above the OK and Cancel buttons.
(Reporter)

Comment 1

18 years ago
Add mail3 keyword so considered for 6.5.
Keywords: mail3

Updated

18 years ago
OS: other → All
Hardware: PC → All
(Reporter)

Comment 2

18 years ago
Are OK and Cancel even valid here? Does Subscribing/Unsubscribing to a newsgroup 
perform the action immediately, or does the user need to click OK for the action 
to occur? Does Cancel really cancel actions the user has performed?  
Yes, ok and cancel are valid.

the changed don't take effect until hit ok.

if you hit cancel, any changes you made will not take effect.  4.x behaved like
this, too.

accepting.  jglick, can you add a link to the spec that shows where the status
bar should be?
Status: NEW → ASSIGNED

Updated

18 years ago
QA Contact: esther → stephend
(Reporter)

Comment 4

18 years ago
No updated spec yet, but I'll attach a picture.
(Reporter)

Comment 5

18 years ago
Created attachment 20208 [details]
Subcribe dialog
shouldn't the ok and cancel buttons be centered?
Keywords: patch
attaching patch, ccing the big boys for a code review.
(Reporter)

Comment 9

18 years ago
As for the centering, we seem to be inconsistent on that one. On Windows, the 
Search Mail Dialog, Filters dialogs, Prefs, Account Settings, Mail Password and 
others have the buttons at the bottom right aligned.  But other dialogs such as 
the "Find in this Page" dialog, have them centered.  On Mac we seem to do right 
aligned. I asked Paul Hangas about this and he said Ben G would be the one to 
ask.

Comment 10

18 years ago
sr=mscott
fixed.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
VERIFIED on Windows 2000120720, Mac 2000120708, and Linux 2000120721.
(Reporter)

Comment 13

17 years ago
Reopening, see attachment below.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
(Reporter)

Comment 14

17 years ago
Created attachment 53272 [details]
current subscribe dialog
(Reporter)

Comment 15

17 years ago
Statusbar at bottom.
http://www.mozilla.org/mailnews/specs/subscribe/images/Subscribe1a.gif
Keywords: mail3 → nsbeta1
(Assignee)

Comment 16

17 years ago
taking off Seth's plate.
Assignee: sspitzer → andreww
Status: REOPENED → NEW
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.7
(Assignee)

Comment 17

17 years ago
waiting for hewitt's changes to <dialog> - to allow for explicit buttons- so I 
can place the statusbar under it.
(Assignee)

Comment 18

17 years ago
Created attachment 60717 [details] [diff] [review]
changes to subscribe.xul
(Assignee)

Comment 19

17 years ago
patch ready for review. 
Keywords: review
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.7 → mozilla0.9.8

Comment 20

17 years ago
Comment on attachment 60717 [details] [diff] [review]
changes to subscribe.xul

r=shliang
Attachment #60717 - Flags: review+
(Assignee)

Updated

17 years ago
Attachment #20262 - Attachment is obsolete: true

Comment 21

17 years ago
Comment on attachment 60717 [details] [diff] [review]
changes to subscribe.xul

sr=hewitt
Attachment #60717 - Flags: superreview+

Comment 22

17 years ago
r=racham.
(Assignee)

Comment 23

17 years ago
fixed
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago17 years ago
Resolution: --- → FIXED
(Reporter)

Comment 24

17 years ago
:-D
I'll spin up a new bug about the lack of vertical bars separating the offline
status indicator from the progress meter, but this bug itself is fixed:

RedHat 7.2 - 2002-01-11-08
Windows 2K - 2002-01-11-03
Mac OS X 10.1.2 - 2002-01-11-08

Verified FIXED.
Status: RESOLVED → VERIFIED

Comment 26

17 years ago
Should dialogs have a statusbar at all? After all they have default padding
which makes the statusbar look odd compared to windows.

Comment 27

17 years ago
Why was this done?  I remember moving it to under the tree a long time ago. It 
looks considerably better attached to the tree, where it is everywhere else in 
the product.  It looks very awkward to not have the OK and Cancel buttons at 
the very bottom of the dialog. And what about the status applies to the OK and 
Cancel buttons?  The status is specific to the population of the tree.  I'm 
looking at this in Classic and I can't imagine seeing a dialog like this in 
another product, so I'm going to be a big jerk and reopen this for a bit more 
discussion :-)
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 28

17 years ago
I could go  either way - this dialog seems like a hybrid anyway..  it's very 
complicated for a modal dialog but do you want the user keeping it up with 
out dismissing it?  

I think this dialog should behave more like the  "search messages"  
dialog - which has no ok/cancel and is non-modal.

Jennifer - what was your original reasoning?


(Assignee)

Comment 29

17 years ago
Created attachment 65517 [details]
view of subscribe in mac classic skin

yeah it does look yukky in mac classic.
(Assignee)

Comment 30

17 years ago
Created attachment 65520 [details]
old subscribe dialog from 4.x w status next to ok cancel
(Reporter)

Comment 31

17 years ago
Created attachment 65521 [details]
4.x - Status bar at bottom

4.x had it at the bottom. It seemed out of place to me, floating above the
buttons. Blake, I thought its location was an error, I didn't know you did this
on purpose. Any window in the product that has a statusbar, the status is at
the very bottom. But this is a dialog so it makes things less clear. If people
agree it looks worse at the bottom, we can switch it back.
(Assignee)

Comment 32

17 years ago
what if we slid the status bar to the left and made room for the ok cancel 
buttons? then the status bar would be closer to it's tree and you'd still 
have the ok cancel buttons in the right place. -- plus remove some of the 
extra cruft from the status bar perhaps.
(Assignee)

Updated

17 years ago
Attachment #65520 - Attachment description: old subscribe dialog from 4.x → old subscribe dialog from 4.x w status next to ok cancel

Comment 33

17 years ago
ok, really outside the box suggestion: could we put the subscribe options into 
the content pane instead of fighting over this awkward dialog?

Advantage H: status bar is where it always is.
I: no modal window

some art to show what i'm thinking:

+----------------------------------------------------------+
|_Mozilla_Messenger____________________________________.O_X|
| File Edit                                                |
>----------------------------------------------------------<
| Back Forward Reload Stop                                 |
>------------------v---------__________--------------------<
|_Folders__________|/Server\|Newsgroups|/Offline\__________|
|-pop.sample.net   ||( )All (*)Search ( )New               |
| o Inbox          ||Newsgroup: [ ne                    ]  |
|-news.aol.com     |>----------------------------v-v-------<
| o aol.im.beta    || Name                      A|o|Count|W|
|>news.mozilla.org<|>----------------------------+-+-------<
| o m.inspector    ||mozilla.derivatives.netscape| | 593 |^|
| o m.mail-news    ||mozilla.mail-news           |.|7108 | |
| o mozilla.reorg  ||mozilla.networking          | |1327 |X|
|------------------||mozilla.networking.qa       | | 479 | |
| Add Server       ||mozilla.newideas            | |6829 |v|
+------------------^--------------------------v------------+
| Status: Loading newsgroup 15/89             |        $ U |
+---------------------------------------------^------------+

Comment 34

17 years ago
> Any window in the product that has a statusbar, the status is at
> the very bottom. But this is a dialog so it makes things less clear. 

Yeah, make your minds up (or get mpt to do it :-)

Personally I prefer the old position of the progressmeter and statusbarpanel,
but since statusbars (will) have resizer grippies they are not suitable for
dialogs, only for windows, although you can just stick the meter and panel into
an hbox.

If you do leave the explicit buttons please put default="true" on the OK button.

Comment 35

17 years ago
> Yeah, make your minds up (or get mpt to do it :-)

Waah, why do I always get the sordid problems?

This problem exists for two reasons. Firstly, bug 57266 isn't fixed. The reason 
this is a dialog in the first place is because the back end doesn't allow you 
to make a change in the Subscribe window and have it immediately reflected in 
any open three-pane windows, and vice versa. This is akin to showing a folder 
window in your file manager (e.g. `My Documents') as a dialog, because some 
other window happens to be showing the same folder at the same time and you 
don't want them to get out of sync. That's not insurmountable; it's just a 
silly architecture bug.

Secondly, faced with the constraint that this window had to be a dialog, 
whoever designed it tried to make it look like a non-dialog window anyway. As I 
said in bug 42090 comment 10, such attempts always end in tears -- the dialog 
in 4.x looked yucky, and the current one is even worse. Document windows and 
utility windows may have status bars; dialogs, as I said in bug 95929 comment 
5, never should.

So, the easy fix for this bug is to get rid of the status bar (fixing bug 95928 
and bug 95929 in the process), to make the dialog really look like a dialog. To 
show progress in loading the folder/group list, use text and/or spinning arrows 
(like we use for sidebar panels) above the top right corner of the list, which 
disappear when the list is complete.

Available groups:                    48 % loaded (\)
+-------------------------------+--------+-------+-+
| Group                         | Unread | Total |H|
+-------------------------------+--------+-------+-+
|                               |        |       |A|

While you're at it, please fix the order of buttons on the Mac -- it should be
( Cancel ) ((  OK  )), not (  OK  ) ( Cancel ).

The long term, and much preferable, solution is to fix bug 57266. Then this 
window could be cleaned up along the lines of bug 83763, and it could indeed 
have a status bar, though it would still probably look better without one.
(Assignee)

Comment 36

17 years ago
MPT: You're right the ok/cancel buttons are indeed reversed. Which that 
alone is enough in my opinion to reopen this and Ill fix that.

ALL: Before we(I) make 10-20 iterations of this in XUL, please come up 
with a screenshot that has approval from relevant parties.  When the 
iterations dust settles from that I'll implement whatever screenie is 
decided upon.  Thanks!
 
Meanwhile I'll go ahead and fix the reversed ok/cancel buttons in a new 
bug and mention that bug here.
(Assignee)

Comment 37

17 years ago
Ok separate bug for ok/cancel issue filed   - patch forthcoming for that 
aspect at least.

http://bugzilla.mozilla.org/show_bug.cgi?id=120799
> ALL: Before we(I) make 10-20 iterations of this in XUL, please come up 
> with a screenshot that has approval from relevant parties.  When the 
> iterations dust settles from that I'll implement whatever screenie is 
> decided upon.  Thanks!

Since we're following her spec, wait for jglick to process all the suggestions 
and update the spec before hacking away.

Comment 39

17 years ago
I agree with Seth. The current spec has it at the bottom.  It looks like the
arguments for different ways of showing status have been given.  It's up to
Jennifer to decide if she wishes to change it.  If not, then we'll close this
bug after Andrew fixes the OK/Cancel buttons on the Mac (unless someone wants to
file a separate bug on that).
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.8 → mozilla0.9.9
(Assignee)

Updated

17 years ago
Keywords: patch, review

Comment 40

17 years ago
approving nomination. Andrew figured out how to put the status bar next to the
buttons. 
Keywords: nsbeta1 → nsbeta1+

Updated

17 years ago
Depends on: 120799
(Assignee)

Comment 41

17 years ago
Created attachment 70241 [details]
new  appearance with latest patch
Attachment #20208 - Attachment is obsolete: true
Attachment #53272 - Attachment is obsolete: true
Attachment #60717 - Attachment is obsolete: true
Attachment #65517 - Attachment is obsolete: true
Attachment #65520 - Attachment is obsolete: true
Attachment #65521 - Attachment is obsolete: true
(Assignee)

Comment 42

17 years ago
Created attachment 70242 [details] [diff] [review]
new patch

the dlgtype buttons in this patch are  hidden  otherwise two sets of ok-cancel
buttons appear because this is a dialog window. If I  remove  them alltogether,
another set of ok  cancel buttons appears due to  the dialog binding.  This
patch allows the status bar and the ok cancel buttons to coexist on the same
horizontal  space and satisfies the need for the status bar to be positioned
close to the tree it  is  associated with, while also allowing the ok cancel
buttons to appear where they need to be as well. this patch also fixes the bug
120799.

Comment 43

17 years ago
Comment on attachment 70242 [details] [diff] [review]
new patch

That --> looks wrong.
The align="right" that was there before looks wrong.
The extra boxes and offline indicator look wrong.
(Assignee)

Comment 44

17 years ago
good feedback ! Looks like I should make a simpler progress meter 
without the extra boxes.

(Assignee)

Comment 45

17 years ago
Created attachment 70390 [details]
classic subscribe panel with v2 changes
Attachment #70241 - Attachment is obsolete: true
Attachment #70242 - Attachment is obsolete: true
(Assignee)

Comment 46

17 years ago
Created attachment 70391 [details]
modern subscribe panel with v2 changes
(Assignee)

Comment 47

17 years ago
Created attachment 70392 [details] [diff] [review]
new v2 patch
(Assignee)

Comment 48

17 years ago
ok - new and improved patch there. Jen - does this look ok?
Keywords: review
(Assignee)

Comment 49

17 years ago
I'm not expecting to get a r/sr by end  of the day, so I'm moving this to 1.0 
but it'll be hopefully ready for checkin as soon as the tree reopens.
Target Milestone: mozilla0.9.9 → mozilla1.0
(Reporter)

Comment 50

17 years ago
Looks much better. Blake, you ok with this solution?

Comment 51

17 years ago
Comment on attachment 70392 [details] [diff] [review]
new v2 patch

Is there any chance that you can turn off the additional status bar elements in
chrome, rather than hiding them in skin? My idea is as follows: Don't overlay
mailWindowOverlay.xul, it's got tons of irrelevant junk in it which produces at
least two JS errors because you're not really a mail window. Instead, just
include the two relevant <statusbarpanel>s, the progressmeter and the status
text.
(Assignee)

Comment 52

17 years ago
ok - I can try that out.  I did try that before with strange results, but ill try it 
w/o the overlay.  

Comment 53

17 years ago
Created attachment 70785 [details]
subscribe.xul but without mailWindowOverlay.xul to get rid of JS errors

Andrew, I hope this is useful, this subscribe dialog has an explicit status
bar.
(Assignee)

Comment 54

17 years ago
Ill take a look at it. I'd need to modify that version to get the ok/cancel
buttons back in the right location.
(Assignee)

Comment 55

17 years ago
Created attachment 71395 [details] [diff] [review]
new v3 patch 

Ok new patch - removes redundant dlgtype - hidden buttons, moves the buttons to
the platformoverlay where they belong, and simplifies the css.
Attachment #70392 - Attachment is obsolete: true
Attachment #70785 - Attachment is obsolete: true

Comment 56

17 years ago
Comment on attachment 71395 [details] [diff] [review]
new v3 patch 

dialogOverlay.xul is completely deprecated.  If you've already converted to
<dialog>, you shouldn't be overlaying dialogOverlay.xul to begin with.

I see that there is a need to extend <dialog> so that we can do custom button
layouts while maintaining platform-specific button arrangements (mainly for mac
vs. windows).  A thought that comes to mind is a new tag called <dialogbuttons>
which you can place within your dialog to get the platform-specific button set.
 Children of this tag would be inserted wherever the platform xbl thinks it
should go.  Andrew, can you hack up this change into dialog.xml ?
Attachment #71395 - Flags: needs-work+

Comment 57

17 years ago
Possible alternative: modify the dialog binding so that <statusbarpanels> are
inserted into an appropriate place: by default before all of the buttons, but on
the mac as children of the <spacer> (after changing it to a <box> of course :-)
(Assignee)

Comment 58

17 years ago
Ok - I like the dialogbuttons idea because it's much more likely to get 
used  in other contexts than progress meters. Plus progress meters are 
more like "content" in a dialog, whereas virtually all dialogs have some 
form of ok/cancel buttons. I'll make another patch and post it here soon. 

 
(Assignee)

Comment 59

17 years ago
Created attachment 75259 [details] [diff] [review]
changes to subscribe.xul

I'm reverting this dialog to it's previous state with some cleanups in order to
fix the button reordering issue.  Making such a huge change to dialog.xml just
for one dialog seemed like a good idea at the time but after pouring over the
code i'm seeing that it'd be way more complicated than simply avoiding having
to have the progressmeter right next to the ok/cancel buttons.	 

So in the interests of	time and getting lots of bugs fixed for machV instead
of spending alot of time on this one alone, I've opted for a simpler patch
which does just about the same thing without having to patch dialog.xml, and
several other files which might destablize  all dialogs for the sake of this
one.   Here's the patch.   Ironically it's alot like the dialog was originally.
Attachment #70390 - Attachment is obsolete: true
Attachment #70391 - Attachment is obsolete: true
Attachment #71395 - Attachment is obsolete: true
(Assignee)

Comment 60

17 years ago
Created attachment 75264 [details]
screenshot of patch in action

screenie

Comment 61

17 years ago
Comment on attachment 75259 [details] [diff] [review]
changes to subscribe.xul

Insert a <separator class="thin"/> (i.e. a small vertical space) between the
tree and the OK/Cancel buttons, and r=hwaara.
Attachment #75259 - Flags: review+

Comment 62

17 years ago
Created attachment 75371 [details] [diff] [review]
Any chance of some extra cleanup?

This patch is basically a diff of my attachment 70785 [details]
but with an extra <row> as per attachment 75259 [details] [diff] [review]
and an extra <separator class="thin"> as per comment 61.
(Assignee)

Comment 63

17 years ago
sure - I added the elements to the final version. 

Comment 64

17 years ago
Comment on attachment 75371 [details] [diff] [review]
Any chance of some extra cleanup?

sr=bienvenu
Attachment #75371 - Flags: superreview+

Comment 65

17 years ago
Comment on attachment 75371 [details] [diff] [review]
Any chance of some extra cleanup?

a=asa (on behalf of drivers) for checkin to the 1.0 trunk and bringing the r=
forward to current patch.
Attachment #75371 - Flags: review+
Attachment #75371 - Flags: approval+
(Assignee)

Comment 66

17 years ago
fix checked in.
Status: REOPENED → RESOLVED
Last Resolved: 17 years ago17 years ago
Resolution: --- → FIXED
Verified FIXED with:

Mac OS 9.2 / 10.1.3 - 2002-03-22-08 Modern and Classic
Linux Mandrake 8.1 - 2002-03-22-08 Modern and Classic
Windows 2000 - 2002-03-22-03 Modern and Classic.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey

Updated

10 years ago
Component: MailNews: Subscribe → MailNews: Message Display
QA Contact: stephend → search
You need to log in before you can comment on or make changes to this bug.