Opened 18 years ago
Closed 17 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 )
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)
Change History (22)
comment:1 by , 18 years ago
by , 18 years ago
| Attachment: | TicketPreview0.9.3.patch added |
|---|
patch to display draft background on ticket preview
comment:2 by , 18 years ago
| Description: | modified (diff) |
|---|---|
| Owner: | changed from to |
| Priority: | low → highest |
| Status: | new → assigned |
I agree that the preview background is better than changing the "Enter" behavior.
comment:3 by , 18 years ago
| Severity: | minor → blocker |
|---|
comment:4 by , 18 years ago
| Severity: | blocker → minor |
|---|
comment:5 by , 18 years ago
| Component: | ticket system → version control |
|---|---|
| Resolution: | → invalid |
| Status: | assigned → closed |
comment:6 by , 18 years ago
| Component: | version control → ticket system |
|---|---|
| Resolution: | invalid |
| Status: | closed → reopened |
WFT?
comment:7 by , 18 years ago
| Type: | defect → enhancement |
|---|
comment:8 by , 18 years ago
| Type: | enhancement → defect |
|---|
comment:9 by , 18 years ago
| Cc: | added |
|---|---|
| Keywords: | kk added |
| Priority: | highest → lowest |
| Type: | defect → task |
test:
comment:10 by , 18 years ago
| Keywords: | kk removed |
|---|---|
| Priority: | lowest → highest |
| Type: | task → defect |
WTF?
This is not a test system!
comment:11 by , 18 years ago
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
comment:12 by , 18 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
Someone seems to think that this is a nice ticket to mess around with :-/
comment:15 by , 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 , 18 years ago
| Owner: | changed from to |
|---|---|
| Status: | reopened → new |
comment:19 by , 18 years ago
comment:20 by , 17 years ago
| Keywords: | preview added |
|---|---|
| Owner: | changed from to |
Eli worked on this topic, in the source:sandbox/pycon/ticket-preview@4990 branch.
comment:21 by , 17 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
And athomas merge that branch in trunk, in r4986.
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.