incubator-bloodhound-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Apache Bloodhound" <bloodhound-...@incubator.apache.org>
Subject Re: [Apache Bloodhound] #372: 'Select All' function on WebAdmin Plugin panel
Date Fri, 08 Feb 2013 07:56:01 GMT
#372: 'Select All' function on WebAdmin Plugin panel
--------------------------+-----------------------
  Reporter:  rjollos      |      Owner:  rjollos
      Type:  enhancement  |     Status:  new
  Priority:  major        |  Milestone:  Release 5
 Component:  ui design    |    Version:  0.4.0
Resolution:               |   Keywords:  jquery
--------------------------+-----------------------

Comment (by rjollos):

 The changeset from [trac 11504] was copied (!JavaScript part only), and
 the following changes made:
  * Replaced selector `table.listing` -> `table.table` to reflect
 !Bloodhound/Trac template differences.
  * Replaced **Enabled** text with a ''select all'' checkbox, rather than
 wrapping it in a `label` and pre-pending the ''select all'' checkbox. The
 **Enabled** text is copied to the element `title` attribute.

 I'm hoping the function of the checkboxes is obvious enough that we don't
 need text in the table header. This was discussed in trac:#9609, but
 ultimately the text was kept. I think the layout doesn't look very nice
 with the text retained, and it looks even worse for Bloodhound than for
 Trac. Removing the text also narrows the column of checkboxes, allowing
 more width for the Component description (though #376 will help with that
 quite a bit as well). We can always add a note somewhere on the page
 describing the function of the checkboxes.

 Similar features have been implemented for the
 [trachacks:AccountManagerPlugin] recently in trachacks:#10745, with more
 similar features forthcoming. These are small steps towards more dynamic
 behavior for the user interface (see #277 for one of the more significant
 changes, and more tickets will be created soon for features outside the
 admin panels which will have more far-reaching impacts on end-users).

 Committed in r1443860. I'll leave the ticket open for a few days for
 comments.

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/372#comment:1>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound (incubating) issue tracker

Mime
View raw message