Closed
Bug 977037
Opened 11 years ago
Closed 11 years ago
Interaction design and visual design for tab-modal dialogs (e.g. HTTP auth)
Categories
(Toolkit :: General, defect)
Tracking
()
VERIFIED
FIXED
mozilla30
People
(Reporter: phlsa, Assigned: mmaslaney)
References
(Blocks 1 open bug)
Details
(Whiteboard: p=5 s=it-30c-29a-28b.3 [qa-])
Attachments
(3 files)
As a prerequisite for bug 613785 and others, we need a generalized design for tab-modal dialogs (that aren't alerts).
Some work in progress can be found here: http://cl.ly/image/0Y2S1G1G3x3A
Michael will take it from here.
Comment 1•11 years ago
|
||
WIP looks good, however I'd propose to move the box directly below the address bar so it somehow looks like the omnibar/dropdown. The reason is that the (sub-)domain is sometimes not providing enough information to make a decision on the prompt (e.g. HTTP login may relate to a folder). Also, some prompts in the mockup don't show any URL at all making it rather hard to know which page requested the prompt (e.g. the "Stay on page?" prompt) leaving only the site title and dimmed page to identify the page. Lastly, there is no way to reach the Identity Panel to verify the sites' identity before entering credentials (HTTP login prompt). This is a no-go – there's already a bug on this somewhere, can't find it right now.
So, please make sure that the full URL is always visible. Some prompts may want to "repeat" the domain name (e.g. HTTP login prompt, to cover the case of a HTTP-auth-required resource inside another site).
Updated•11 years ago
|
Updated•11 years ago
|
Reporter | ||
Comment 2•11 years ago
|
||
(In reply to Florian Bender from comment #1)
> WIP looks good, however I'd propose to move the box directly below the
> address bar so it somehow looks like the omnibar/dropdown. The reason is
> that the (sub-)domain is sometimes not providing enough information to make
> a decision on the prompt (e.g. HTTP login may relate to a folder). Also,
> some prompts in the mockup don't show any URL at all making it rather hard
> to know which page requested the prompt (e.g. the "Stay on page?" prompt)
> leaving only the site title and dimmed page to identify the page. Lastly,
> there is no way to reach the Identity Panel to verify the sites' identity
> before entering credentials (HTTP login prompt). This is a no-go – there's
> already a bug on this somewhere, can't find it right now.
>
> So, please make sure that the full URL is always visible. Some prompts may
> want to "repeat" the domain name (e.g. HTTP login prompt, to cover the case
> of a HTTP-auth-required resource inside another site).
That is actually something Michael and I talked about yesterday. The URL bar will be visible (and functional) in the final design :)
Comment 3•11 years ago
|
||
Awesome, thanks :)
IMHO, without bug 647010 the domain below the the TLD (and IPs if no domain) should be emphasized, e.g.
* https://www.example.com/show_bug.cgi?id=977037 -> emphasize "example.com"
* https://myintranet/show_bug.cgi?id=977037 -> emphasize "myintranet"
* https://63.245.215.20/show_bug.cgi?id=977037 -> emphasize "63.245.215.20"
* https://[2620:101:8008:5::2:1]/show_bug.cgi?id=977037 -> emphasize "[2620:101:8008:5::2:1]"
Reporter | ||
Comment 5•11 years ago
|
||
Bug 741050 (about the downloads dialog) could also benefit from this approach.
Reporter | ||
Comment 6•11 years ago
|
||
(In reply to papalowa from comment #4)
> IMHO, without bug 647010 the domain below the the TLD (and IPs if no domain)
> should be emphasized, e.g.
> * https://www.example.com/show_bug.cgi?id=977037 -> emphasize "example.com"
> * https://myintranet/show_bug.cgi?id=977037 -> emphasize "myintranet"
> * https://63.245.215.20/show_bug.cgi?id=977037 -> emphasize "63.245.215.20"
> * https://[2620:101:8008:5::2:1]/show_bug.cgi?id=977037 -> emphasize
> "[2620:101:8008:5::2:1]"
Makes sense! Since we already do that in the location bar, it should fit in quite nicely.
Updated•11 years ago
|
Whiteboard: p=0
Reporter | ||
Comment 7•11 years ago
|
||
Here's a design spec from Michael (still WIP as far as I know)
Reporter | ||
Comment 8•11 years ago
|
||
And some mockups of other use cases that should use the same kind of dialog:
http://cl.ly/image/0p3b113u1t2o
http://cl.ly/image/0Z1d0E0D0s31
Updated•11 years ago
|
Status: NEW → ASSIGNED
Whiteboard: p=0 → p=5 s=it-30c-29a-28b.3
Updated•11 years ago
|
Whiteboard: p=5 s=it-30c-29a-28b.3 → p=5 s=it-30c-29a-28b.3 [qa-]
Assignee | ||
Comment 9•11 years ago
|
||
Assignee | ||
Comment 10•11 years ago
|
||
Assignee | ||
Comment 11•11 years ago
|
||
Spec CSS
.secondary-button-text {
font-size: 12.5px;
line-height: 18.75 px;
font-family: "Segoe UI";
color: rgb( 51, 51, 51 );
text-align: left;
}
.secondary-button {
border-style: solid;
border-width: 1px;
border-color: rgb( 193, 193, 193 );
border-radius: 2px;
background-color: rgb( 251, 251, 251 );
height: 26px;
}
.secondary-button:hover {
background-color: rgb( 255, 255, 255 );
}
.secondary-button:active {
background-color: rgb( 200, 200, 200 );
}
.primary-button-text {
font-size: 12.5px;
line-height: 18.75px;
font-family: "Segoe UI";
color: rgb( 255, 255, 255 );
text-align: left;
}
.primary-button {
border-radius: 2px;
background-color: rgb( 116, 191, 67 );
height: 26px;
}
.primary-button:hover {
background-color: rgb( 108, 178, 62 );
}
.primary-button:hover:active {
background-color: rgb( 100, 164, 58 );
}
.promp h1 {
font-size: 23px;
line-height: 30 px;
font-family: "Segoe UI";
color: rgb( 51, 51, 51 );
text-align: left;
}
.promp-tag {
font-size: 12.5px;
line-height: 18.75 px;
font-family: "Segoe UI";
color: rgb( 230, 96, 0 );
text-align: left;
}
.promp h2 {
font-size: 12.5px;
line-height: 18.75 px;
font-family: "Segoe UI";
font-weight: bold;
color: rgb( 51, 51, 51 );
text-align: left;
}
.promp-text {
font-size: 12.5px;
line-height: 18.75 px;
font-family: "Segoe UI";
color: rgb( 51, 51, 51 );
text-align: left;
}
.prompt-container {
border-style: solid;
border-width: 1px;
border-color: rgb( 23, 24, 26 );
background-color: rgba( 255, 255, 255, 0.969 );
opacity: 0.2;
box-shadow: 0px 3px 5px 0px rgb( 23, 24, 26 );
}
.controls-container {
background-color: rgb( 241, 241, 241 );
box-shadow: 0px -1px 0px 0px rgb( 213, 213, 213 );
}
.input {
border-style: solid;
border-width: 1px;
border-color: rgb( 187, 187, 187 );
background-color: rgb( 255, 255, 255 );
}
.input:active {
border-color: rgb( 0, 150, 221 );
}
Reporter | ||
Comment 12•11 years ago
|
||
Note that the buttons at the bottom should be the other way around on Windows.
Windows: Sign in | Cancel
OS X/Linux: Cancel | Sign in
That way we meet the platform conventions.
Assignee | ||
Comment 13•11 years ago
|
||
Good catch, Philipp!
Assignee | ||
Updated•11 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Status: RESOLVED → VERIFIED
Updated•11 years ago
|
Target Milestone: --- → mozilla30
Comment 15•11 years ago
|
||
To whoever implements this, we already use this footer on UITour tooltips (info panels), the identity popup (maybe a slightly different colour), bookmark/history panels and on the menu panel; we also use the button styling on UITour tooltips. Please make re-useable styles (e.g. classes) and/or defines so we aren't hard-coding these styles in 4+ places. It seems like we should have a <footer> element or similar that provides the footer styling in any panel and provides the proper negative margins so that it reaches the edges of default panels.
Updated•11 years ago
|
No longer blocks: fxdesktopbacklog
Flags: firefox-backlog+
Reporter | ||
Updated•11 years ago
|
Comment 16•7 years ago
|
||
Quick question: Does this also fix not being able to navigate anywhere else, that is, a different tab or an extension (e.g. a Password Manager) while a pop-up for Basic Authentication is showing? This is something that bugs me for a *very* long time now. Would be nice if it finally got fixed :-)
Comment 17•5 years ago
|
||
@thomas.ebert: no, what you are writing about is that the Basic Auth pop-up is currently window-modal. And the bug for this is bug 613785 .
You need to log in
before you can comment on or make changes to this bug.
Description
•