Bug (done?)

Ctrl-Klick in firefox/#5246

Summary

done?
May 22, 2007
100%
May 23, 2007 / xl
Feb 20, 2011 / guest
tino
 

Attached files

No files uploaded
 
This task does not have any text yet.
Doubleclick here to add some.

Issue report

Major
Always
firefox (linux/ubuntu)
0.08
Ctrl-Klick on Link in Task List.
Link is opened in same window.
In firefox Ctrl-Klick opens a link in a new window.
 

8 Comments

pixtur:Confirmed...

5 years ago

This is really a pain in the ass. Somehow the assigned click-handler also grabs the CTRL-Click events but removes the modifier key.

I still have no clue how to fix this.


xl:Perhaps

5 years ago

...just de-assign the click handler for links. Is jquery used? I begin to like this.

tino:Not sure...

5 years ago

I'll take a look at this...

pixtur:yeah, it's jquery.

5 years ago

The code can be found in js/listFunctions.js:

The normal clicks are already de-assigned for normal links.

    /**
    * prevent normal links from being overwritten by row toggling
    */
    $('table.list a').click
    (
        function(e)
        {
            document.location.href= this.href;
            return false
        }
    );

tino:It looks like this...

5 years ago

I played a bit and this is my first result

from js/listFunctions.js

    $('table.list a').click(function(e) {
        if( e.ctrlKey )
        {
        	document.location.href = gBrowser.addTab(this.href);
        	return false;
        }
        else 
        {
            document.location.href= this.href;
            return false;
        }
    });

xl:solution

5 years ago

one comment to this:
i think this works for me, but not for everyone.

if i understand it right, this emulates firefox standard behabiour.
but ctrl-click in firefox is configurable (open new tab in foreground or in background).

the "real" solution might be to switch off streber click handling for links completely and let firefox do the job.

pixtur:Antwort auf solution

5 years ago

Could you elaborate a bit more on this? How can this be done...

xl:let the browser do its work

5 years ago

what i wanted to say:
we should not build a complex click handler but pass on to the browser what is to the browser. (what ctrl-click does is a matter of browser config, not of our javascript!)

i looked a bit into js/listFunctions.js.
the "trouble" code you cited in yeah, it's jquery., what is it good for anyway?
it is commented "prevent normal links from being overwritten by row toggling", but it looks like row toggling is done server side, not javascript.

so the solution woud be to just delete cited code.