Project

General

Profile

Actions

Patch #20370

closed

Lighter colors for journal details in issue history

Added by Go MAEDA over 8 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Category:
UI
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

This change makes easier to distinguish details (attachments, changes of statuses, assignees, ...) and notes in a issue history.

screenshot


Files

journal-detail-background-color.diff (711 Bytes) journal-detail-background-color.diff patch Go MAEDA, 2015-07-19 08:44
sample-19097.png (113 KB) sample-19097.png screenshot Go MAEDA, 2015-07-19 08:49
FieldChanges.png (82.8 KB) FieldChanges.png budo kaiman, 2015-10-02 13:14
FilesAdded.png (115 KB) FilesAdded.png budo kaiman, 2015-10-02 13:14
patch-20151005.diff (1.55 KB) patch-20151005.diff Go MAEDA, 2015-10-04 18:03
Redmine_Pathch-20151005.png (97 KB) Redmine_Pathch-20151005.png budo kaiman, 2015-10-05 13:53
patch-20151005_lighter.diff (1.55 KB) patch-20151005_lighter.diff budo kaiman, 2015-10-05 17:24
Redmine_patch-20151005_lighter.png (103 KB) Redmine_patch-20151005_lighter.png budo kaiman, 2015-10-05 17:26
Actions #1

Updated by Go MAEDA over 8 years ago

  • Target version set to Candidate for next major release
Actions #2

Updated by Michael Goegel over 8 years ago

+1
Even with round corners to prevent injuries :o)

Actions #3

Updated by Go MAEDA over 8 years ago

  • Target version changed from Candidate for next major release to 3.2.0
Actions #4

Updated by Jean-Philippe Lang over 8 years ago

Do we really want to highlight the journal details? Most of the time, people want them to be less visible. I would rather fade them out with something like color:#aaa.
I'd like to have more feedback on this before making any change.

Actions #5

Updated by budo kaiman over 8 years ago

Jean-Philippe Lang wrote:

Do we really want to highlight the journal details? Most of the time, people want them to be less visible. I would rather fade them out with something like color:#aaa.
I'd like to have more feedback on this before making any change.

I agree that it should be less visible opposed to highlighting it, and your suggestion looks good when only fields have changed

But when files are attached, it doesn't seem to make much difference

Actions #6

Updated by Jean-Philippe Lang over 8 years ago

budo kaiman wrote:

I agree that it should be less visible opposed to highlighting it, and your suggestion looks good when only fields have changed

But when files are attached, it doesn't seem to make much difference

We can choose a lighter blue for links. For example:

.journal ul.details {color:#aaa;}
.journal ul.details a {color:#8da3cc;}
.journal ul.details a:hover {color:#c61a1a;}
Actions #7

Updated by Go MAEDA over 8 years ago

Thanks for feedback. I will make an updated patch.

Actions #8

Updated by Go MAEDA over 8 years ago

Updated the patch.

  • From the viewpoint of readability, I think '#aaa' is too pale. Chose '#808080' (same color with activity description) for journal details and changed color of main text from '#484848' to '#333'.
  • Increased the separation between journal details and a note.

FYI: GitHub uses '#767676' and #333'. example: https://github.com/phusion/passenger/issues/1622

Actions #9

Updated by budo kaiman over 8 years ago

I agree with the decision to use '#808080' and I think the patch looks much better.

Edit: On second thought, I think '#808080' might be a tad too dark. I tried a few values and found '#959595' to work well.

Actions #10

Updated by budo kaiman over 8 years ago

Attaching a patch with some lighter values which I think work better and are still readable.

Actions #11

Updated by Jean-Philippe Lang over 8 years ago

  • Subject changed from Set background color of journal details in a issue history to Lighter colors for journal details in issue history
  • Status changed from New to Closed
  • Assignee set to Jean-Philippe Lang

Patch committed in r14686, thanks.

Actions

Also available in: Atom PDF