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 #11605 (closed)

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

webclient plugin fixes

Reported by: wmoore Owned by: wmoore
Priority: major Milestone: OMERO-4.4.10
Component: Web Version: n.a.
Keywords: n.a. Cc:
Resources: n.a. Referenced By: n.a.
References: n.a. Remaining Time: n.a.
Sprint: n.a.

Description

In preparing the webclient plugin docs https://github.com/openmicroscopy/ome-documentation/pull/523, a few RFEs came up:

  • Shouldn't need plugin_tab_index, since we should be able to determine that from the index of the dom element.
  • plugin_enabled / tab_enabled and load_plugin_content / load_tab_content. Since these were first developed for tab plugin *then* for centre panel - need to replace the tab* methods (but continue to support them.

Change History (5)

comment:1 Changed 6 years ago by wmoore

  • Resolution set to fixed
  • Status changed from new to closed

comment:2 Changed 6 years ago by Will Moore <will@…>

(In [9137035a4fc56ca9d759dcf47f4ae138a168daab/ome.git] on branch develop) Replace 'load_tab_content' with 'load_plugin_content'. See #11605

comment:3 Changed 6 years ago by Will Moore <will@…>

(In [47658d60037332eb30b2b61bf39d132672b48133/ome.git] on branch develop) Remove need for plugin_index setting. See #11605

comment:4 Changed 6 years ago by Will Moore <will@…>

(In [5599aa9279931bb5edc8d29b4375ab5c32c3d94f/ome.git] on branch dev_4_4) Replace 'load_tab_content' with 'load_plugin_content'. See #11605

comment:5 Changed 6 years ago by Will Moore <will@…>

(In [6339ff76c605481c964908d8ce83f7e70e1922cf/ome.git] on branch dev_4_4) Remove need for plugin_index setting. See #11605

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.75140 sec.)

We're Hiring!