Closed Bug 25071 Opened 25 years ago Closed 24 years ago

'View Background Image' in non-image context menu.

Categories

(SeaMonkey :: General, enhancement, P3)

enhancement

Tracking

(Not tracked)

VERIFIED FIXED
Future

People

(Reporter: bugzilla, Assigned: gerv)

References

Details

Attachments

(2 files)

Just like you have the "view image" it could be nice if there where also a "view background image" if a webpage/table has a background image.
IMHO, there are hundreds of contextual menu items that could each be convenient to a tiny fraction of users --- but if every one of them were added to an application, it would then be impossible to find the ones that most people do in fact need. Can you think of any reasons why this particular one will be useful to a significant number of them? --- Reassigning to German, who spec'd the context menu items.
Assignee: shuang → german
Severity: normal → enhancement
Summary: Ability to "view background image" → [enh]Ability to "view background image"
Target Milestone: M20
Have to agree with Eli here, I too am afraid of context menu overkill. How do you envision using this feature?
Summary: [enh]Ability to "view background image" → Ability to "view background image"
Moving all UE/UI bugs to new component: User Interface: Design Feedback UE/UI component will be deleted.
Component: UE/UI → User Interface: Design Feedback
Mainly I use View Image to see the file name of the image (in NC 4.x it appears in the context menu). It would be nice to have this same functionality for background images. No need to add another item to the context menu, just make View Image load the background image if the mouse is over a background image.
QA Assigning non-confidential New/Assigned User Interface: Design Feedback bugs to Matthew Thomas (mpt@mailandnews.com). Matthew Thomas is now the QA owner for the User Interface: Design Feedback component. (Bugs that involve UI issues in the Netscape-branded Mozilla browser should continue be QA assigned to elig@netscape.com.)
QA Contact: elig → mpt
This wouldn't really be context menu overkill, because (a) it would only appear in the context menu for the empty part of a page (and that menu isn't very large already), and (b) there would be no other way to get to this command (short of an intrepid excursion into the Page Info window). You wouldn't need the whole suite of image context menu commands, just `View Image' -- the rest of the image-related items would then be in the context menu for the image when it was loaded in the window by itself.
Yse agreed: it is an overkill. Set resolution to WON'TFIX for this version.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → WONTFIX
Reopening and setting to Future. I don't think it's acceptable to require the user to hunt through Page Info just to be able to open the background image. I've seen other browsers with this menu item; and if we're not going to have `Set as Wallpaper' like previous versions of Mozilla did, I think having `View Background Image' instead is quite reasonable.
Status: RESOLVED → REOPENED
OS: Windows 98 → All
Hardware: PC → All
Resolution: WONTFIX → ---
Target Milestone: M20 → Future
I do not have any major objections against adding it, its just one more thing that I would rate low in terms of usefulness for the average Netscape audience, compared to other things that need fixing...
I think 'View background image' is not something that the average Netscape audience would worry about whether it was in the context menu. IMO, This feature is only reasonably interesting for a web-design audience, or for those more than causal web user. I propose reassigning this bug to ben, the mozilla UI owner, so he can add this to his 'ongoing discussions bin'
Assignee: german → ben
Status: REOPENED → NEW
Actually `Save Background' is in Netscape's own context menu spec <http://bugzilla.mozilla.org/showattachment.cgi?attach_id=18576>. Replacing that with `View Background Image' would allow users to view the background unobscured, and then save it using `File'>`Save', and print it using `File'>`Print', etc etc. If there is only one background-image-related command in the context menu, this should be it, because it provides indirect access to all the others.
It would also be neat if "[Load/Show] Background Image" were shown for background images that haven't been loaded. See also bug 35130 for that.
Summary: Ability to "view background image" → 'Save Background As...' in non-image context menu.
Maybe this and other more obscure links could be traversed through Page Info or some similar metadata viewer? Can you do that currently?
I disagree with the `updating' of the summary from having `View Background Image' to `Save Background As ...'. If you have `Save Background As ...', you can not print the background, copy the image to the clipboard, copy the image URL, add a bookmark for the image, print the image, e-mail it to someone, etc. If you have `View Background Image' instead, you can do all of those things once you have the image open by itself in the window, *and* you are still be able to save it by choosing `File' > `Save' (or `Save Image' in the context menu for the image itself).
See also bug 40230.
Chaning the qa contact on these bugs to me. MPT will be moving to the owner of this component shortly. I would like to thank him for all his hard work as he moves roles in mozilla.org...Yada, Yada, Yada...
QA Contact: mpt → zach
Assignee: ben → gervase.markham
OK, I have a working implementation of this. Taking bug from Ben :-) I have replaced the (un-hooked-up) Save Background As... code with View Background Image, to appear instead of View Image in appropriate circumstances. This currently uses the View Image handling code, and so opens in a new window. Questions: - Should this menu item only appear if you click "directly" on the background (for some value of directly)? Currently it appears whenever there's a background image, and you haven't directly clicked on another image. Is this the right behaviour? - Are there any instances where the value of "background" will not be an image? Gerv
i had two good testcases for this stuff... one was nsa and one was my high school. i still believe the only good solution is abandoning our flat menu. i'll find the testcases and probably post some browser comparisons eventually (or sooner if someone reminds me later)
timeless: Was that comment meant for this bug? I don't understand... Gerv
Testcases (pages with backgrounds and complex content): http://www.mcps.k12.md.us/schools/rockvillehs/ http://www.nsa.gov/programs/employ/index.html Gerv
I'm going to wait until my other context menu patch is checked in before sorting this one, so I don't have to keep disentangling the two :-) Resummarising to reflect the actual implementation. Gerv
Summary: 'Save Background As...' in non-image context menu. → 'View Background Image' in non-image context menu.
*** Bug 40230 has been marked as a duplicate of this bug. ***
Depends on: 75338
Attached patch Patch - attempt 1 (deleted) — Splinter Review
I lost the old code, so I had to rewrite it, and am attaching the patch for safekeeping. I'm looking for r=, but may not be actively chasing it for a while. Still, if someone wants to, that would be great :-) Gerv
r=timeless
Keywords: approval, patch
Attached patch New un-rotted patch (deleted) — Splinter Review
sr=blizzard by email; checked in by bbaetz. Gerv
Status: NEW → RESOLVED
Closed: 25 years ago24 years ago
Resolution: --- → FIXED
yeps.. verified in build 20010626
Status: RESOLVED → VERIFIED
*** Bug 125218 has been marked as a duplicate of this bug. ***
I have no problem with the 'View Background Image' solution. Only in my version (Build ID 2002020406) it doesn't open in a new window. This way I have to go back and it has to render the possibly complex page again.
Component: User Interface Design → Browser-General
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: