Go back to the group list is rendered inline with Yes, delete

RESOLVED FIXED in Bugzilla 3.0

Status

()

Bugzilla
Administration
--
minor
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

3.1.1
Bugzilla 3.0
Bug Flags:
approval +
approval3.0 +

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
Created attachment 278279 [details] [diff] [review]
insert </p>

<h2>Confirmation</h2>

  <p>Do you really want to delete this group?</p>
    <p><b>You must check all of the above boxes or correct the
          indicated problems first before you can proceed.</b></p>

  <p><input type="submit" value="Yes, delete">
  <input type="hidden" name="action" value="delete">
  <input type="hidden" name="group" value="26">

  <input type="hidden" name="token" value="XEEZBritpl">
</form>

Go back to the <a href="editgroups.cgi">group list</a>.
Attachment #278279 - Flags: review?(vladd)

Updated

11 years ago
Assignee: administration → timeless
Flags: approval?
Flags: approval3.0?
OS: Mac OS X → All
Hardware: Macintosh → All
Target Milestone: --- → Bugzilla 3.0

Updated

11 years ago
Attachment #278279 - Flags: review?(vladd) → review+

Comment 1

11 years ago
Wouldn't it be nicer to keep them on the same line?

Updated

11 years ago
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Flags: approval?
Flags: approval3.0?
Flags: approval3.0+
Flags: approval+

Comment 2

11 years ago
I suppose that decision could be made somewhere else. Here, lacking </p> is a bug.

Comment 3

11 years ago
(In reply to comment #2)
> I suppose that decision could be made somewhere else. Here, lacking </p> is a
> bug.

Only with HTML 4 Strict, not Transitional where </p> is optional. So I wouldn't approve this patch based on this argument only.
(Assignee)

Comment 4

11 years ago
Comment on attachment 278279 [details] [diff] [review]
insert </p>

mozilla/webtools/bugzilla/template/en/default/admin/groups/delete.html.tmpl 	1.13

BUGZILLA-3_0-BRANCH:
mozilla/webtools/bugzilla/template/en/default/admin/groups/delete.html.tmpl 	1.10.2.1
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.