Closed
Bug 79990
Opened 24 years ago
Closed 13 years ago
shift+contextmenu-"Reload" does not Force-Reload
Categories
(SeaMonkey :: UI Design, defect, P5)
SeaMonkey
UI Design
Tracking
(Not tracked)
RESOLVED
WORKSFORME
Future
People
(Reporter: mbabcock-mozilla, Unassigned)
References
()
Details
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux 2.2.19 i686; en-US; rv:0.9) Gecko/20010505
BuildID: 2001050521
If one shift-clicks the reload button in the browser, one gets a forced refresh
header in one's request to force proxies to recheck documents. If one
right-clicked the document then held shift while clicking reload from the
context menu in 0.8, it did the same thing. In 0.9, it does not ... it does a
normal refresh.
Reproducible: Always
Steps to Reproduce:
n/a
Comment 1•24 years ago
|
||
probably xp apps gui features....
Did this really work in 0.8???? (I have a hard time believing we actually test
for "shift" on context menu clicks).
Assignee: asa → blakeross
Component: Browser-General → XP Apps: GUI Features
QA Contact: doronr → sairuh
Reporter | ||
Comment 2•24 years ago
|
||
I've used this feature in every version of Netscape I've had, and with Mozilla
and a client's website I've been working on in 0.8 and 0.81, this function
worked as well as far as I can remember. The fact that I'm somewhat dismayed
that it no longer works indicates to me that it did before ;-) but I can check
an old copy of 0.8 to check.
Reporter | ||
Comment 3•24 years ago
|
||
I tested against 0.8. What has changed is that in 0.8, all reloads were forced
refreshes (reported by Squid as TCP_CLIENT_REFRESH_MISS). In 0.9 it seems that
quick refreshes (has this changed? no? ok.) have been implemented for all
reloads except shift-clicking the reload button on the toolbar.
So in 0.8, shift wasn't detected on the context menu but it wasn't relevant
because all reloads behaved the way a shift-reload should.
Updated•24 years ago
|
QA Contact: sairuh → claudius
Comment 4•24 years ago
|
||
Makring NEW.
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
Keywords: regression,
ui
OS: Linux → All
Hardware: PC → All
Comment 5•24 years ago
|
||
claudius <washes hands> since the failure is only within the context menu, bouncing back.
sorry for spam
QA Contact: claudius → sairuh
Comment 6•24 years ago
|
||
so not a regression, enhancement request.
Severity: minor → enhancement
Keywords: regression → helpwanted
Comment 8•23 years ago
|
||
Uh...Matthew? Shift-clicking on the menuitem?
(To tell the truth, we can't do or really shouldn't be doing that.)
Reporter | ||
Comment 9•23 years ago
|
||
If we "shouldn't" be doing this, then how am I to force a refresh on a page
within a frame?
Currently, I have to do a "view only current frame" then do a shift-reload on
the title-bar, then hit "back".
Comment 10•23 years ago
|
||
we have a "reload frame" context menuitem - perhaps make it work as a shift reload?
Updated•23 years ago
|
Summary: Right-click reload & reload button not the same → shift+click on "reload" in context menu to force-reload frame
Reporter | ||
Comment 11•23 years ago
|
||
Does anyone see any detrimental effects to setting the right-click (context
menu) "reload frame" option to always doing a conditional verification of the
page content?
Is there a bug entry of any form for how Mozilla will be handling caching of
pages in general? That would be a good place for a lot of this discussion. It
seems somewhat obvious that Mozilla ought to be a good Internet citizen and do
conditional refreshes based on Etags, etc. using HTTP/1.1 where possible, so the
refresh cases I'm dealing with would dissapear if that was done properly on page
and frame reloads in the first place.
To be terribly verbose, could we perhaps have multiple items with different
meanings? "Conditional Refresh" vs. "Reload from Server" (but shorter ;-).
Comment 12•23 years ago
|
||
I agree with Michael and Doron that it would make sense for the `Reload Frame'
context menu item to always do a forced refresh of the frame -- since there
isn't another convenient way to offer forced frame refreshes. (Try saying that
ten times fast.) We're hardly going to be killing the world's proxy caches by
doing that only for this obscure menu item.
I disagree with Michael on everything else. :-) Bugzilla is not a good place
for a lot of *any* discussion; and separate `Conditional Refresh' and `Reload
from Server' context menu items would be more confusing than useful.
Reporter | ||
Comment 13•23 years ago
|
||
Thank-you Matthew for that support -- I was simply thinking outloud about
whether it was possible or even desirable to properly indicate that the two
refreshes (the toolbar vs. the context menu) don't do the same thing.
Re: discussion on bugzilla; when there are quicklinks between threads on the
newsgroups and bugzilla, then discussions will happen in the newsgroups. Until
then, people have their discussions in bugzilla (where, you're correct, they
ought not be).
Updated•23 years ago
|
Target Milestone: --- → Future
Comment 14•23 years ago
|
||
*** Bug 92885 has been marked as a duplicate of this bug. ***
Comment 15•23 years ago
|
||
I do think that this is a bug - I expect that any menu items and buttons with
the same name also behave the same. Expecially in such a case, where the
"malfunction" is not directly discoverable, but with consequences.
Severity: enhancement → normal
Reporter | ||
Comment 16•23 years ago
|
||
It _is_ a bug that functionality is available but only under certain conditions
and hidden from the user. I've filed bug 98922 to request the additional menu item.
Comment 17•23 years ago
|
||
Clarifying SUMMARY a bit more (hopefully)
Summary: shift+click on "reload" in context menu to force-reload frame → hift+contextmenu-"Reload" does not Force-Reload
Updated•23 years ago
|
Summary: hift+contextmenu-"Reload" does not Force-Reload → shift+contextmenu-"Reload" does not Force-Reload
Updated•23 years ago
|
Status: NEW → ASSIGNED
Comment 18•23 years ago
|
||
taking bug
Assignee: blaker → cbiesinger
Status: ASSIGNED → NEW
Keywords: helpwanted
Priority: -- → P1
Target Milestone: Future → mozilla0.9.9
Comment 19•23 years ago
|
||
damn, blake was right...
event.shiftKey is always false for a <menuitem>
I filed bug 126189 about this issue.
Depends on: 126189
Comment 21•23 years ago
|
||
*** Bug 137724 has been marked as a duplicate of this bug. ***
Updated•22 years ago
|
Priority: P1 → P5
Comment 22•22 years ago
|
||
is this even needed?
Comment 23•22 years ago
|
||
doron, do you mean that this bug should not be fixed, or that bug 126189 does
not need to be fixed for this one? If the latter, why not? If I can't find out
if the shift key is pressed, I can't do a force reload if it's pressed.
Reporter | ||
Comment 24•22 years ago
|
||
It was suggested at one point that the context-menu reload always be a
forced-refresh. Is this a valid idea?
Comment 25•22 years ago
|
||
I am talking about the this bug, if someone wants to shift reload, use the
toolbar button.
(how long till someone mentions browsing without the toolbar?)
So perhaps make the reload frame always a shift-reload?
Comment 26•22 years ago
|
||
doron, this bug is about user expectations. For example, people tell other
people to "SHIFT-reload". They don't say "but don't use the context menu".
Comment 27•22 years ago
|
||
I have learned that UI patches are not wanted in this project. reassigning to
default owner.
Assignee: cbiesinger → blaker
QA Contact: sairuh → paw
Updated•20 years ago
|
Product: Core → Mozilla Application Suite
Comment 28•20 years ago
|
||
*** Bug 290729 has been marked as a duplicate of this bug. ***
Updated•18 years ago
|
Assignee: bross2 → guifeatures
QA Contact: pawyskoczka
Comment 29•16 years ago
|
||
Filter "spam" on "guifeatures-nobody-20080610".
Assignee: guifeatures → nobody
QA Contact: guifeatures
Comment 30•13 years ago
|
||
Closing as WORKSFORME. Fixed as part of Bug 529240.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•