Opened 12 years ago

Closed 4 years ago

#5029 closed enhancement (patchwelcome)

Make skipForm="true" by default for DnD operations.

Reported by: Eugene Lazutkin Owned by: Eugene Lazutkin
Priority: low Milestone: 1.13
Component: DnD Version: 0.9
Keywords: reviewed Cc:
Blocked By: Blocking:

Description (last modified by Eugene Lazutkin)

I am getting too many invalid bug reports, when people assume that draggable form should be editable by default. It makes sense, and it was not done this way before because I didn't want to break the original behavior. Now I think this change is unlikely to break anybody's code --- who wants draggable but non-editable forms anyway?

Change History (11)

comment:1 Changed 12 years ago by Eugene Lazutkin

Status: newassigned

comment:2 Changed 12 years ago by Eugene Lazutkin

Description: modified (diff)

comment:3 Changed 12 years ago by Adam Peller

Milestone: 1.0.11.0.2

comment:4 Changed 12 years ago by Adam Peller

Milestone: 1.0.21.0.3

comment:5 Changed 11 years ago by dylan

Milestone: 1.0.31.1

comment:6 Changed 11 years ago by bill

Milestone: 1.11.2

Move all milestone 1.1 tickets to 1.2, except for reopened tickets and tickets opened after 1.1RC1 was released.

comment:7 Changed 11 years ago by Eugene Lazutkin

Milestone: 1.2future

Moving all open ticketd to the future.

comment:8 Changed 7 years ago by ben hockey

Keywords: needsreview added
Priority: highlow

i'm trying to identify stale tickets. if there is a need to keep this ticket open, please replace the "needsreview" keyword with "reviewed". if there is no need to keep this ticket open then please close it.

comment:9 in reply to:  8 Changed 7 years ago by Eugene Lazutkin

Replying to neonstalwart:

i'm trying to identify stale tickets. if there is a need to keep this ticket open, please replace the "needsreview" keyword with "reviewed". if there is no need to keep this ticket open then please close it.

Unfortunately it was not reviewed. And we don't have a protocol on how to handle such behavioral changes.

I just closed #11128, where user complained exactly about skipForm: false issue thinking its a bug --- so the issue is still there.

comment:10 Changed 6 years ago by bill

Keywords: reviewed added; needsreview removed

comment:11 Changed 4 years ago by dylan

Milestone: future1.12
Resolution: patchwelcome
Status: assignedclosed

Given that no one has shown interest in creating a patch in the past 8 years, I'm closing this as patchwelcome.

Note: See TracTickets for help on using tickets.