On Mon, Jan 12, 2009 at 4:22 PM, David Duymelinck <[email protected]>wrote:

>
> I saw there were a lot of browser commands so why not add copy-tab :
>
>
> CmdUtils.CreateCommand({
>    name: "copy-tab",
>    description: "Copies current tab",
>    preview: function( pblock ) {pblock.innerHTML=this.description;},
>    execute: function() {
>        var win = CmdUtils.getWindowInsecure();
>        Utils.openUrlInBrowser(win.location.href);
>    }
> });
>
>
> David
>

Great idea, however, we need to make sure this works even if people have
"new pages should be opened in" == "a new window"
Currently, it will throw an error (I'm not quite sure why though), but even
without that, the copy tab would open in a new window and not a tab, because
Utils.openUrlInBrowser respects that setting

-- cers / Christian Sonne

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"ubiquity-firefox" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/ubiquity-firefox?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to