Closed
Bug 431070
Opened 17 years ago
Closed 16 years ago
"Auto-Update" needs to pop up a Sparkle window, not point to Reload
Categories
(Camino Graveyard :: Product Site, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: alqahira, Assigned: phiw2)
References
()
Details
Attachments
(2 files, 2 obsolete files)
|
698 bytes,
patch
|
Details | Diff | Splinter Review | |
|
22.81 KB,
image/png
|
phiw2
:
review+
|
Details |
Right now, hovering the auto-update button causes the descriptive text to point right at the Reload button, which is both odd and wrong.
Sam and I talked about making hovering that also pop up a Sparkle window (or part of a Sparkle window) that the descriptive text can point to instead of pointing at Reload.
However, this probably requires us getting a designer or having a lot of time on our hands. We do need to do it, though.
| Assignee | ||
Comment 1•17 years ago
|
||
1. That will require a (small) change in the html, is that OK ? One way or another, the image of the Sparkle window need to inserted. Adding the image of the Sparkle window in the html (hidden) is easiest, and fastest for hover response.
2. The Sparkle window is quite big, just slightly smaller than the Camino window currently used on the site.
option 1: put the whole Sparkle window, slightly offset from top and left of Camino window and add the descriptive text on top of it. Not difficult, iExploder will probably ffff but nothing that can't be solved.
option 2: crop the Sparkle window, how only top part
I'll see what I can do, prolly by tomorrow, Tuesday local time
Comment 2•17 years ago
|
||
(In reply to comment #1)
> 2. The Sparkle window is quite big, just slightly smaller than the Camino
> window currently used on the site.
> option 1: put the whole Sparkle window, slightly offset from top and left of
> Camino window and add the descriptive text on top of it. Not difficult,
> iExploder will probably ffff but nothing that can't be solved.
> option 2: crop the Sparkle window, how only top part
Is option 3 (scaling the Sparkle window) a possibility?
| Reporter | ||
Comment 3•17 years ago
|
||
My thought was either to crop it or to use the "you're up to date" window (which is not as effective at getting the message across, though).
Scaling would distort the UI/perspective too much, I think.
Adding a bit to the html should be OK; we aren't terribly rigid with ensuring 100% the same structure on every page. Everything starting with <div id="preview"> until just before <div id="footer"> is the page itself.
Thanks for taking a look at this, philippe :) Assigning to you for now, then.
Assignee: samuel.sidler → phiw
| Assignee | ||
Comment 4•17 years ago
|
||
Hmm. Scaling is an option I guess. Getting the window to look good is another thing.
And, wouldn't it look strange, UI wise?
| Assignee | ||
Comment 5•17 years ago
|
||
A very quick comp in photoshop
The Sparkle window is 70% of its real size.
(obviously, the colour of the window controls and buttons is not correct)
Comment 6•17 years ago
|
||
I'm in favor of either cropping it (with some style/fade out maybe?) or using the up-to-date window like we do on the Features page.
I'd rather not scale it.
| Reporter | ||
Comment 7•17 years ago
|
||
While we're thinking about bending reality, we could also edit that nib to let us use a smaller minimum size and then make the window shorter. Not sure if that helps enough to be worthwhile.
I'd rather not scale it, either; that scaled window freaks me out.
| Assignee | ||
Comment 8•17 years ago
|
||
Yeah, that scaled thing looks a bit (ahem) odd. I posted it for discussion, but it doesn't get my vote.
My current thought is a cropped Sparkle window, clipped just below the 'release notes' line, with a slight fade out effect. Position would be towards the bottom of the main window, with the bubble pointing to the Sparkle Window title.
(on a related note, am I wrong to assume that IE6 win is not in the list of supported browsers ? The front page of cb-o looks rather cr*p)
| Assignee | ||
Comment 9•17 years ago
|
||
Work in progress:
http://dev.l-c-n.com/camino/cbo/cbo.html
| Reporter | ||
Comment 10•17 years ago
|
||
philippe, were you waiting on feedback from us, or did you just have to get back to paying work? :)
The overall WIP idea looks good, but there are a couple of things that distract me a little bit:
1) I think that the Sparkle window needs to be taller before the fade-out (i.e., so that the fade occurs over the Find bar or status bar--somewhere a little less noticeable).
2) I also think that maybe the Sparkle window should be a bit higher up on the page, not so much for the window itself, but rather so the bubble for the descriptive text is not so far down. If we move the bubble to point at the Sparkle window title as you mentioned in comment 8 (does that end up not looking as good as the current positioning?), it might not be necessary to move the window up.
As for IE6, I know we have some CSS stuff for it, but I have no idea whether it was supposed to have worked at one point and we messed it up or if it was always broken.
| Assignee | ||
Comment 11•17 years ago
|
||
(In reply to comment #10)
> philippe, were you waiting on feedback from us, or did you just have to get
> back to paying work? :)
Yah, paying stuff got in the way.
> The overall WIP idea looks good, but there are a couple of things that distract
> me a little bit:
>
> 1) I think that the Sparkle window needs to be taller before the fade-out
> (i.e., so that the fade occurs over the Find bar or status bar--somewhere a
> little less noticeable).
Making the Sparkle window increases the file size quite a bit. I could of course remove as much transparency as possible and use a very small shadow. PNG 32 is not so friendly to file size.
> As for IE6, I know we have some CSS stuff for it, but I have no idea whether it
> was supposed to have worked at one point and we messed it up or if it was
> always broken.
>
I'll have a look at fixing as much as possible. I suspect the PNG files don't play too well there (that browser has no support for PNG 32). The hover effects don't work either (li:hover is not supported).
IE 7 also needs some love, I think. But that is minimal surgery.
| Assignee | ||
Comment 12•17 years ago
|
||
New version, based on the comments in comment 11
http://dev.l-c-n.com/camino/cbo/cbo2.html
The SU window image is a bit large in filesize (28kb), I'll try to reduce that some more.
The file includes some fixes for IE 6 and 7.
I'l file a separate bug for those fixes.
| Assignee | ||
Comment 13•17 years ago
|
||
(In reply to comment #12)
> The file includes some fixes for IE 6 and 7.
> I'l file a separate bug for those fixes.
for the record, I filed bug 433218 for those.
| Assignee | ||
Comment 14•17 years ago
|
||
With a bit more polish
http://dev.l-c-n.com/camino/cbo/cbo3.html
Sam, mind if you have a look at this ?
Some small explanation:
That attached diff is only for the changes to the (main) stylesheet (and it doesn't include the change for bug 433218).
The html of the page needs one modification: inserting the su-window image. I put it as part of the 'auto-update' li, just after the span
the new code looks like this:
Auto-Update <span class="update">When a new release is available, Camino will offer you the option of downloading and installing it.</span> <img src="/path/to/image/suwindow3.png" width="584" height="165" alt="Software update window" id="suwindow" /></a></li>
obviously, replace /path/to/image/ with the actual path to the image
I'll attach the image next.
Attachment #318096 -
Attachment is obsolete: true
Attachment #320653 -
Flags: review?(samuel.sidler)
| Assignee | ||
Comment 15•17 years ago
|
||
Attachment #320654 -
Flags: review?(samuel.sidler)
| Reporter | ||
Comment 16•17 years ago
|
||
This all looks good to me (but Sam has the final say here); thanks, philippe!
Keywords: helpwanted
Updated•17 years ago
|
Attachment #320653 -
Flags: review?(samuel.sidler) → review?(isaac.melan)
Comment 17•17 years ago
|
||
Comment on attachment 320654 [details]
the su-window image
r=me if you update it for 1.6.3 so it doesn't have b4 in it. Isaac will review the other patch.
Attachment #320654 -
Flags: review?(samuel.sidler) → review+
| Assignee | ||
Comment 18•17 years ago
|
||
(In reply to comment #17)
> (From update of attachment 320654 [details])
> r=me if you update it for 1.6.3 so it doesn't have b4 in it. Isaac will review
> the other patch.
>
'k. I'll post a new image as soon as possible (Weekend job).
| Assignee | ||
Comment 19•17 years ago
|
||
Updated per comment 17, carrying over Sam's r+
Now 4kb smaller (there is less text in the window).
In context:
http://dev.l-c-n.com/camino/cbo/cbo4.html
Attachment #320654 -
Attachment is obsolete: true
Attachment #336228 -
Flags: review+
| Reporter | ||
Comment 20•16 years ago
|
||
:(
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WONTFIX
| Reporter | ||
Comment 21•16 years ago
|
||
Comment on attachment 320653 [details] [diff] [review]
css changes for the display of the SU window
Apparently WONTFIXing a bug doesn't cancel reviews :P
Attachment #320653 -
Flags: review?(isaac.melan)
You need to log in
before you can comment on or make changes to this bug.
Description
•