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

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

<pre>
.journal ul.details {color:#aaa;}
.journal ul.details a {color:#8da3cc;}
.journal ul.details a:hover {color:#c61a1a;}
</pre>
--------------------------------------------------------------------------------
Thanks for feedback. I will make an updated patch.
--------------------------------------------------------------------------------
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
--------------------------------------------------------------------------------
I agree with the decision to use '#808080' and I think the patch looks much better.

!{width: 70%}Redmine_Pathch-20151005.png!

Edit: On second thought, I think '#808080' might be a tad too dark. I tried a few values and found '#959595' to work well.
--------------------------------------------------------------------------------
Attaching a patch with some lighter values which I think work better and are still readable.

!{width: 70%}Redmine_patch-20151005_lighter.png!
--------------------------------------------------------------------------------
Patch committed in r14686, thanks.
--------------------------------------------------------------------------------