Edgewall Software

Ticket #2888 (closed defect: fixed)

Opened 3 years ago

Last modified 22 months ago

Misleading behavior - preview Ticket change

Reported by: Felix Collins Owned by: ecarter
Priority: highest Milestone: 0.11
Component: ticket system Version: 0.9
Severity: minor Keywords: preview
Cc: kkiffe77@…

Description (last modified by mgood) (diff)

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

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

Change History

Changed 3 years ago by anonymous

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.

Changed 3 years ago by Felix Collins

patch to display draft background on ticket preview

Changed 3 years ago by mgood

  • owner changed from jonas to mgood
  • priority changed from low to highest
  • status changed from new to assigned
  • description modified (diff)

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

Changed 2 years ago by anonymous

  • severity changed from minor to blocker

Changed 2 years ago by mgood

  • severity changed from blocker to minor

Changed 2 years ago by anonymous

  • status changed from assigned to closed
  • resolution set to invalid
  • component changed from ticket system to version control

Changed 2 years ago by eblot

  • status changed from closed to reopened
  • resolution invalid deleted
  • component changed from version control to ticket system

WFT?

Changed 2 years ago by anonymous

  • type changed from defect to enhancement

Changed 2 years ago by anonymous

  • type changed from enhancement to defect

Changed 2 years ago by anonymous

  • cc kkiffe77@… added
  • keywords kk added
  • type changed from defect to task
  • priority changed from highest to lowest

test:

Changed 2 years ago by eblot

  • keywords kk removed
  • priority changed from lowest to highest
  • type changed from task to defect

WTF?

This is not a test system!

Changed 2 years ago by anonymous

  • status changed from reopened to closed
  • resolution set to fixed

Changed 2 years ago by cmlenz

  • status changed from closed to reopened
  • resolution fixed deleted

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

Changed 2 years ago by anonymous

Why not block anonymous from changing properties then?

Changed 2 years ago by anonymous

kmjmkh

Changed 2 years ago by anonymous

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 ;-)

Changed 2 years ago by anonymous

  • owner changed from mgood to lirong@…
  • status changed from reopened to new

Changed 2 years ago by mgood

  • owner changed from lirong@… to mgood

Why are you trying to change the owner?

Changed 2 years ago by mgood

  • milestone set to 0.11

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

Changed 2 years ago by cboos

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...

Changed 22 months ago by cboos

  • keywords preview added
  • owner changed from mgood to ecarter

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

Changed 22 months ago by cboos

  • status changed from new to closed
  • resolution set to fixed

And athomas merge that branch in trunk, in r4986.

Add/Change #2888 (Misleading behavior - preview Ticket change)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
to The owner will change from ecarter. Next status will be 'closed'
 
Note: See TracTickets for help on using tickets.