Opened on 03/25/2014 at 10:01:56 AM

Closed on 03/25/2014 at 04:19:43 PM

Last modified on 03/25/2014 at 04:29:41 PM

#211 closed change (fixed)

[trac] Make priority/milestone/component fields visible to everybody

Reported by: trev Assignee: trev
Priority: P2 Milestone:
Module: Infrastructure Keywords:
Cc: Blocked By:
Blocking: Platform:
Ready: yes Confidential: no
Tester: Verified working: yes
Review URL(s):

http://codereview.adblockplus.org/5769066073554944/

Description

Background

Currently we hide fields that the user isn't allowed to edit. That has the side-effect however that these fields are also hidden in the issue summary or issue listing. This information is important however and needs to be shown to everybody.

What to change

Show fields like priority/milestone/component in the issue summary but keep them hidden when the issue is created and the user doesn't have the permission to change them.

Attachments (0)

Change History (8)

comment:1 Changed on 03/25/2014 at 10:19:05 AM by trev

Looking into the blackmagic plugin, there really seem to be only two options: "hide" will hide the field everywhere, "disable" will show it but it will also show the edit field (which will be disabled, with the label crossed out). The following CSS code should do the job however:

  label[for^="field-"] > strike,
  [id^="field-"][disabled]
  {
    display: none;
  }

Given that blackmagic doesn't remove the table cells but only the text within, the job done by this CSS code isn't any worse.

comment:2 Changed on 03/25/2014 at 11:14:37 AM by trev

  • in_progress changed from 0 to 1
  • Owner set to trev
  • Status changed from new to assigned

comment:3 Changed on 03/25/2014 at 11:52:58 AM by trev

  • Review URL(s) modified (diff)
  • Status changed from assigned to reviewing

comment:4 Changed on 03/25/2014 at 02:44:08 PM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:5 Changed on 03/25/2014 at 03:15:22 PM by philll

Looks good apart from http://trac.edgewall.org/ticket/11560 when regarding the review_URL field.

comment:6 Changed on 03/25/2014 at 03:34:11 PM by trev

  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening due to issues with hiding the "Review URL(s)" field as well as a missing dependency in Puppet.

comment:7 Changed on 03/25/2014 at 04:19:43 PM by trev

  • Resolution set to fixed
  • Status changed from reopened to closed

comment:8 Changed on 03/25/2014 at 04:29:41 PM by philll

  • Verified working set

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
 
Note: See TracTickets for help on using tickets.