Ticket #10657 (closed enhancement: fixed)

Opened 6 months ago

Last modified 3 months ago

Hide the 'Order' column

Reported by: rjollos Assigned to: rjollos
Priority: normal Component: AdminEnumListPlugin
Severity: normal Keywords:
Cc: jun66j5 Trac Release:

Description

When this plugin is enabled (i.e. when JS is enabled in the browser), it would be nice to hide the 'order' column since it no longer serves much purpose. With at least Trac 1.0, there seems to be a conflict: when items are reordered with the select box, the order updates correctly but it is no longer possible to reorder the items by drag and drop.

Possibly related, the following errors are seen in the JS error console on Chrome 21 when re-ordering with either the selects or by drag and drop: 

Uncaught TypeError: Object function (a,b){return new p.fn.init(a,b,c)} has no method 'curCSS'

It would also be useful to provide a tooltip indicating that the items can be reordered by drag and drop.

Attachments

adminenumlistplugin-r12474.diff (2.8 kB) - added by jun66j5 on 12/25/12 14:24:13.

Change History

11/26/12 11:04:57 changed by rjollos

I intend to provide a patch for this ticket and #10656 soon.

12/08/12 08:58:47 changed by rjollos

  • owner changed from nonplus to rjollos.
  • cc set to jun66j5.
  • status changed from new to assigned.

(in reply to: ↑ description ; follow-up: ↓ 4 ) 12/08/12 11:22:34 changed by rjollos

Replying to rjollos:

Possibly related, the following errors are seen in the JS error console on Chrome 21 when re-ordering with either the selects or by drag and drop:

The errors are due to a compatibility issue between jQuery UI < 1.8.22 and jQuery 1.8. Since Trac provides jQuery UI 1.8.21, we'll have to implemented a workaround. I'll commit this shortly.

(in reply to: ↑ 3 ) 12/08/12 19:59:09 changed by rjollos

Replying to rjollos:

The errors are due to a compatibility issue between jQuery UI < 1.8.22 and jQuery 1.8. Since Trac provides jQuery UI 1.8.21, we'll have to implemented a workaround. I'll commit this shortly.

This is a bit more complex than I expected, so I've opened a dedicated ticket: #10688.

(follow-up: ↓ 6 ) 12/09/12 12:58:09 changed by rjollos

The more I think about this, we might want to have an option hide_selects to preserve the previous behavior. We'll need to pass the boolean value to the script, and in order to preserve compatibility with 0.11 we can't use add_script_data, so we'll have to follow Jun's suggestions for passing data to a script in Trac 0.11.

I haven't tested yet with Trac 0.11 though, and there doesn't seem to be any reason that this plugin should work with Trac 0.11. Trac 0.11.0 provides jQuery 1.2.3, and Trac 0.11.7 provides jQuery 1.2.6. This plugin currently packages jQuery UI 1.7.2, which requires jQuery 1.3.

I've come across situations, such as with the DateFieldPlugin, where the functionality utilized in jQuery UI just happens to work with a version of jQuery that is older than what is officially supported, so maybe this is okay. More testing is needed. If the plugin is not working with Trac 0.11, then we can just drop support for that Trac version and move on. If someone needs it later, we/they can work to make it backward compatible.

If the plugin does work with Trac 0.11, we have several options:

  • Make a release for v1.1 of the plugin and drop support for < Trac 0.12 while implementing this feature.
  • Add the option and do the work of passing the boolean value to the script and aim for full compatibility with Trac 0.11.
  • Don't add the hide_selects option because it is deemed not worth the trouble.

Feedback on these comments is welcome and appreciated.

(in reply to: ↑ 5 ) 12/21/12 22:52:16 changed by rjollos

Replying to rjollos:

* Add the option and do the work of passing the boolean value to the script and aim for full compatibility with Trac 0.11.

I have decided to go this route.

12/21/12 22:56:19 changed by rjollos

(In [12462]) Refs #10657:

  • Added option to hide the column of select elements.
  • Script is only added to pages with enum panels, rather than all admin pages.
  • Bumped version to 2.0dev.
  • Updated changelog for recent changes.
  • Added 3-Clause BSD license to source.

12/21/12 23:39:25 changed by rjollos

(In [12463]) Refs #10657, #10695:

  • closest doesn't exist until jQuery 1.3, so its use was replaced with parents.
  • prop is recommended for setting the DOM element statesdisabled and checked in jQuery 1.6+. prop is aliased to attr in version of jQuery that it doesn't exist.

In Trac 0.11.0 (jQuery 1.2.3) the indeterminate state of the checkbox is not seen. It may not be supported in earlier versions of jQuery.

12/25/12 14:24:13 changed by jun66j5

  • attachment adminenumlistplugin-r12474.diff added.

12/25/12 14:27:50 changed by jun66j5

I tried adminenumlistplugin r12474 and found some issues.

  1. Should use BoolOption instead of Option for hide_selects. BoolOption is available since Trac 0.10.
  2. If Python 2.4, req.path_info.startswith(self._panels, ...) raises the following error. 
    TypeError: coercing to Unicode: need string or buffer, tuple found
  3. If Python 2.4, 'true' if self.hide_selects else 'false' raises SyntaxError.
  4. If hide_selects is true with non en_US locale, Order header cell is not hidden.

Here is adminenumlistplugin-r12474.diff tested with Trac 0.11, 0.12.4 and 1.0-stable.

12/25/12 15:18:43 changed by rjollos

(In [12475]) Refs #10657: Added COPYING file that should have been included in [12462].

12/25/12 15:26:21 changed by rjollos

(In [12476]) Refs #10657: Use BoolOption rather than Option. Restored Python 2.4 compatibility. Hiding the order column did not work for locales other than en_US. Patch by Jun Omae (jun66j5).

12/25/12 15:27:18 changed by rjollos

Thanks for the patch Jun!

12/25/12 17:09:51 changed by rjollos

t:#10994 has been opened for including a reworked version of this plugin in the Trac core.

(follow-up: ↓ 15 ) 12/27/12 15:20:43 changed by rjollos

It looks like I could do a small bit of refactoring: 

$group_checkbox.prop('checked', true).prop('indeterminate', false);

-> 

$group_checkbox.prop({'checked': true, 'indeterminate': false});

and likewise for similar lines with multiple calls to prop.

Overall, do you think it is ready for the official release of version 2.0 Jun?

(in reply to: ↑ 14 ) 01/04/13 10:38:14 changed by jun66j5

Sorry for my late response.

Overall, do you think it is ready for the official release of version 2.0 Jun?

Looks good to me.

02/17/13 03:55:53 changed by rjollos

  • status changed from assigned to closed.
  • resolution set to fixed.

02/17/13 04:18:56 changed by rjollos

(In [12620]) Refs #10657: Updated changelog.

Add/Change #10657 (Hide the 'Order' column)




Change Properties
Action