Warning: Can't synchronize with repository "(default)" (/home/git/ome.git does not appear to be a Git repository.). Look in the Trac log for more information.
Notice: In order to edit this ticket you need to be either: a Product Owner, The owner or the reporter of the ticket, or, in case of a Task not yet assigned, a team_member"

Task #8879 (closed)

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

Bug: Ghost sub-menus with dropdowns

Reported by: saloynton Owned by: atarkowska
Priority: critical Milestone: OMERO-4.4
Component: Web Version: n.a.
Keywords: testing,phase1,safari,ie Cc: web-team@…
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: 0.0d
Sprint: 2012-06-19 (17)

Description

Ghost sub-menus with dropdowns - intermittent also found in Mac-Safari. With both disabled items - (greyed out) in IE9 (Win 7) and also active in Safari (OSX). Also happening in Safari on Mac with active menus (not greyed out).

  • OMERO.web 4.3.3-2195-6bc6897-ice33-b151.
  • Windows 7 - IE9

Attachments (2)

Bug- Ghost sub-menus.pdf (150.6 KB) - added by saloynton 12 years ago.
Ghost submenus OSX Firefox.png (937.7 KB) - added by rkferguson 12 years ago.

Download all attachments as: .zip

Change History (14)

Changed 12 years ago by saloynton

comment:1 Changed 12 years ago by wmoore

  • Owner changed from web-team@… to wmoore
  • Priority changed from critical to minor

This is purely a CSS (not functional) issue, right?

comment:2 Changed 12 years ago by atarkowska

  • Cc web-team@… added; rkferguson removed
  • Keywords testing phase1 safari ie added

comment:3 Changed 12 years ago by cxallan

  • Owner changed from wmoore to rkferguson

comment:4 Changed 12 years ago by wmoore

This is due to the opacity of disabled top-menu items being set to 0.5. Because the sub-menus are nested, their opacity is also set to 0.5.

li.jstree-contextmenu-disabled {
    opacity: 0.5;
}

in components/tools/OmeroWeb/omeroweb/webgateway/static/3rdparty/jquery.jstree/themes/default/style.css

comment:5 Changed 12 years ago by jburel

  • Sprint changed from 2012-05-22 (15) to 2012-06-05 (16)

Moved from sprint 2012-05-22 (15)

comment:6 Changed 12 years ago by jburel

  • Sprint changed from 2012-06-05 (16) to 2012-06-19 (17)

Moved from sprint 2012-06-05 (16)

comment:7 Changed 12 years ago by atarkowska

  • Owner changed from rkferguson to atarkowska
  • Remaining Time set to 0.3

comment:8 Changed 12 years ago by atarkowska

  • Priority changed from minor to critical

comment:9 Changed 12 years ago by atarkowska

That is fixed by enabling element that contains submenu in the main line but disabling individuals of submenu.
For example ccp submenu in the main line is enable but each element in submenu are disabled:

// see #8879 config["ccp"]["_disabled"] = true; 
config["ccp"]["submenu"]["copy"]["_disabled"] = true;
config["ccp"]["submenu"]["unlink"]["_disabled"] = true;
config["ccp"]["submenu"]["paste"]["_disabled"] = true;
config["ccp"]["submenu"]["cut"]["_disabled"] = true;

comment:10 Changed 12 years ago by atarkowska

  • Status changed from new to accepted

comment:11 Changed 12 years ago by atarkowska

  • Remaining Time changed from 0.3 to 0
  • Resolution set to fixed
  • Status changed from accepted to closed

Any better solution welcome. Closing for now.

comment:12 Changed 12 years ago by rkferguson

Retested today on Win 7 IE9 (worst affected), OSX all of Firefox, Safari and Camino

All still affected but appears confined to the sub-menus of greyed out menu items - does not occur with sub-menus of available items. See screenshot.

As the only menu that is actually affected in the data tree is the orphaned images - the top item - all the rest have no greyed out items with sub-menus - this must be a low priority, but would be nice to have it fixed at some stage.

Suggest moving it to list of UI changes to be addressed in future.

Changed 12 years ago by rkferguson

Note: See TracTickets for help on using tickets. You may also have a look at Agilo extensions to the ticket.

1.3.13-PRO © 2008-2011 Agilo Software all rights reserved (this page was served in: 0.68568 sec.)

We're Hiring!