Opened 12 months ago

Last modified 5 months ago

#20791 reopened feature request

Add tolerance: 'pointer' option to widgets

Reported by: alexvorn2 Owned by:
Priority: normal Milestone: Awaiting Review
Component: Widgets Version: 3.3.2
Severity: normal Keywords: has-patch dev-feedback
Cc: mdhansen@…

Description

I made a conclusion that it's more easy to sort the widgets in a sidebar if is with tolerance: 'pointer' option enabled...

please consider adding this to sortable function of the widgets.

Attachments (2)

20791-1.diff​ (425 bytes) - added by jkudish 12 months ago.
first pass at adding tolerance: pointer to the widgets sortable js
20791.2.diff​ (413 bytes) - added by MikeHansenMe 7 months ago.
file rename

Download all attachments as: .zip

Change History (11)

comment:1 follow-up: ↓ 2   jkudish — 12 months ago

  • Keywords needs-patch 2nd-opinion removed
  • Resolution set to invalid
  • Status changed from new to closed

Looks like the latest copy of trunk already has tolerance: pointer, as per: ​https://core.trac.wordpress.org/browser/trunk/wp-admin/js/widgets.dev.js#L153

comment:2 in reply to: ↑ 1 ; follow-up: ↓ 3   SergeyBiryukov — 12 months ago

  • Keywords needs-patch 2nd-opinion added
  • Resolution invalid deleted
  • Status changed from closed to reopened

Replying to jkudish:

Looks like the latest copy of trunk already has tolerance: pointer

It's in droppable() function. This ticket is about sortable().

comment:3 in reply to: ↑ 2   jkudish — 12 months ago

Replying to SergeyBiryukov:

It's in droppable() function. This ticket is about sortable().

My mistake, should have looked closer :(

jkudish — 12 months ago

first pass at adding tolerance: pointer to the widgets sortable js

comment:4   alexvorn2 — 11 months ago

  • Version changed from 3.3.2 to 3.4

comment:5   DrewAPicture — 11 months ago

  • Version changed from 3.4 to 3.3.2

Version field tracks when the feature was first requested.

comment:6   SergeyBiryukov — 11 months ago

  • Keywords has-patch dev-feedback added; needs-patch 2nd-opinion removed

MikeHansenMe — 7 months ago

file rename

comment:7   MikeHansenMe — 7 months ago

Updated the file to apply patch to widgets.js instead of widgets.dev.js now that production js is minified.

comment:8   MikeHansenMe — 7 months ago

  • Cc mdhansen@… added

comment:9   alexvorn2 — 5 months ago

  • Severity changed from minor to normal
Note: See TracTickets for help on using tickets.