123624 - click() event does not fire on menuitems in browser

Status: REOPENED

(Core :: XUL, defect)

Description

[shrirang khanzode]

var save = document.getElementById("ok");
save.click();

Shouldn't this generate a click event on the ok button ?

Comment 1

[Fabian Guisset]

not according to the DOM2 HTML spec, and not according to our current
implementation ;-) however perhaps NS4 allowed it...

Comment 2

[jag (Peter Annema)]

It currently works, though.

Comment 3

[jag (Peter Annema)]

I was assuming we're talking about XUL buttons.

Comment 4

[John Morrison]

Attachment: er, it does, no?

<?xml version="1.0"?> 
<?xml-stylesheet href="chrome://communicator/skin/" type="text/css"?>
<window 
  xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
<script>
function routeClick() {
  var save = document.getElementById("ok");
  save.click();
}
</script>
  <button id="ok" label="id ok" onclick="alert('id ok onclick handler');"/>
  <button id="ko" label="other" onclick="routeClick();"/>
</window>

Comment 5

[jag (Peter Annema)]

Yeah, it does with my recent checkin that generates mousedown/up/click events
_and_ calls DoCommand from nsXULElement::Click().

WFM

Comment 6

[shrirang khanzode]

ugh..I did not explain myself properly..sorry abt that.:(

In my testcase below, click event does not work on any menuitem. I am trying to 
deliver a click on the 'Go Offline' menuitem without user intervention. But it 
does not work.  Showed this to jrgm, he also tried to create and deliver an 
event to this thing..but it does not work. 

----copy------

<title>js test</title>
<script type="application/x-javascript">

function getXULDocument()
{
  var windowMediator = 
Components.classes["@mozilla.org/rdf/datasource;1?name=window-mediator"]
                               
.getService(Components.interfaces.nsIWindowMediator);
  var lastWindow = windowMediator.getMostRecentWindow(null);

 
  return lastWindow.document;
}

function openMenu()
{
 
  netscape.security.PrivilegeManager.enablePrivilege("UniversalXPConnect");
  var doc = getXULDocument();
  var fileMenu = doc.getElementById("menu_File");
  var menuBoxObject = 
fileMenu.boxObject.QueryInterface(Components.interfaces.nsIMenuBoxObject);
  menuBoxObject.openMenu(true); 
  var goOffline = doc.getElementById("offlineGoOfflineCmd");
  goOffline.click();
}
  
   
</script>

<p>
  <ul>
    <li><a href="" onclick="openMenu()">open menu</a></li> 
  </ul>
</p>


----end--------

Comment 7

[jag (Peter Annema)]

Reopening then

Comment 8

[jag (Peter Annema)]

-> 1.2

Comment 9

[timeless]

shouldn't you trigger command()?