Edgewall Software
Modify

Opened 19 years ago

Closed 18 years ago

#2888 closed defect (fixed)

Misleading behavior - preview Ticket change

Reported by: Felix Collins Owned by: Eli Carter
Priority: highest Milestone: 0.11
Component: ticket system Version: 0.9
Severity: minor Keywords: preview
Cc: kkiffe77@… Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Matthew Good)

When one edits a ticket and hits the enter key, the behavior is to preview the ticket. The preview condition is not obvious. Unless the description is changed the only indication is the "#preview" on the end of the URL…

I propose that the preview button be shifted to reside next to the Description edit box and that the default action for the page (on hitting enter) is to submit changes.

I've entered this as a defect as I consider it a design defect. An alternative view is that it is an enhancement.

Attachments (1)

TicketPreview0.9.3.patch (1.1 KB ) - added by Felix Collins 19 years ago.
patch to display draft background on ticket preview

Download all attachments as: .zip

Change History (22)

comment:1 by anonymous, 19 years ago

Reading a bit more about this issue it becomes apparent that it is to deal with browser s treating enter in different ways. I different proposal to that stated above is to put a Draft background behind the ticket when preview is hit.

by Felix Collins, 19 years ago

Attachment: TicketPreview0.9.3.patch added

patch to display draft background on ticket preview

comment:2 by Matthew Good, 18 years ago

Description: modified (diff)
Owner: changed from Jonas Borgström to Matthew Good
Priority: lowhighest
Status: newassigned

I agree that the preview background is better than changing the "Enter" behavior.

comment:3 by anonymous, 18 years ago

Severity: minorblocker

comment:4 by Matthew Good, 18 years ago

Severity: blockerminor

comment:5 by anonymous, 18 years ago

Component: ticket systemversion control
Resolution: invalid
Status: assignedclosed

comment:6 by Emmanuel Blot, 18 years ago

Component: version controlticket system
Resolution: invalid
Status: closedreopened

WFT?

comment:7 by anonymous, 18 years ago

Type: defectenhancement

comment:8 by anonymous, 18 years ago

Type: enhancementdefect

comment:9 by anonymous, 18 years ago

Cc: kkiffe77@… added
Keywords: kk added
Priority: highestlowest
Type: defecttask

test:

comment:10 by Emmanuel Blot, 18 years ago

Keywords: kk removed
Priority: lowesthighest
Type: taskdefect

WTF?

This is not a test system!

comment:11 by anonymous, 18 years ago

Resolution: fixed
Status: reopenedclosed

comment:12 by Christopher Lenz, 18 years ago

Resolution: fixed
Status: closedreopened

Someone seems to think that this is a nice ticket to mess around with :-/

comment:13 by anonymous, 18 years ago

Why not block anonymous from changing properties then?

comment:14 by anonymous, 18 years ago

kmjmkh

comment:15 by anonymous, 18 years ago

Well, of course this must be a nice ticket to mess around with, as it is available from within the Sandbox(!). Let me explain how I got here:

  • On the Trac homepage (http://trac.edgewall.org/) I clicked on the "SandBox" link in the text at the bottom of the page
  • In the SandBox I clicked on the "View Tickets" tab
  • Still assuming I'm in the SandBox, I clicked on "All Active Tickets"
  • This ticket here was on top of the list, so I just clicked it
  • Result: I got to this ticket while still assuming being _in_the_SandBox_ (!)

I guess that everbody else and their dog also assumed this…

ciao, daniel ;-)

comment:16 by anonymous, 18 years ago

Owner: changed from Matthew Good to lirong@…
Status: reopenednew

comment:17 by Matthew Good, 18 years ago

Owner: changed from lirong@… to Matthew Good

Why are you trying to change the owner?

comment:18 by Matthew Good, 18 years ago

Milestone: 0.11

I'll try to get this in the next milestone.

comment:19 by Christian Boos, 18 years ago

Also, in the WorkFlow branch, the preview is much nicer than it currently is, and it's much more obvious that you're actually seeing the preview. So I'd say after the WorkFlow merge, we should reconsider this ticket and see if there's still a need to do something about this…

comment:20 by Christian Boos, 18 years ago

Keywords: preview added
Owner: changed from Matthew Good to Eli Carter

Eli worked on this topic, in the source:sandbox/pycon/ticket-preview@4990 branch.

comment:21 by Christian Boos, 18 years ago

Resolution: fixed
Status: newclosed

And athomas merge that branch in trunk, in r4986.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Eli Carter.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Eli Carter to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.