Defect #20882

% done: progress bar blocked at 80 in the issue list

Added by Jérôme BATAILLE almost 6 years ago. Updated over 5 years ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:Jean-Philippe Lang% Done:

0%

Category:UI
Target version:3.3.0
Resolution:Fixed Affected version:

Description

It's because of a css right padding in /public/stylesheets/rtl.css :

/***** Tables *****/
table.list td {padding-left:0px; padding-right:10px;}

A fix :

table.list table.progress td {
    padding-right:0px;
}

redmine_issues_done_ok.png (20.7 KB) Jérôme BATAILLE, 2016-03-19 01:42

redmine_issues_done_stuck_80.png (20.7 KB) Jérôme BATAILLE, 2016-03-19 01:42

i-got-it.png (23.1 KB) Toshi MARUYAMA, 2016-03-26 06:38

Associated revisions

Revision 15291
Added by Jean-Philippe Lang over 5 years ago

% done: progress bar blocked at 80 in the issue list (#20882).

Patch by Jérôme BATAILLE.

History

#1 Updated by Jérôme BATAILLE almost 6 years ago

Sorry, I can't fix the description :

A fix to put in public/stylesheets/application.css :

table.list table.progress td {
    padding-right:0px;
}

#2 Updated by Toshi MARUYAMA almost 6 years ago

  • Description updated (diff)

#3 Updated by Toshi MARUYAMA almost 6 years ago

  • Category deleted (Issues)
  • Status changed from New to Needs feedback

rtl.css is for Right-to-left (Arabic etc.), right?

#4 Updated by Jérôme BATAILLE over 5 years ago

Toshi MARUYAMA wrote:

rtl.css is for Right-to-left (Arabic etc.), right?

You're right ! the same instruction to override is in application.css :

table.list td {text-align:center; vertical-align:top; padding-right:10px;}

#5 Updated by Jérôme BATAILLE over 5 years ago

Hi, can you check this simple patch ?

#6 Updated by Go MAEDA over 5 years ago

  • Category set to UI

@Jérôme BATAILLE:
Could you please attach screenshots? (before and after). After I see it, I will set target version.

#7 Updated by Jérôme BATAILLE over 5 years ago

Go MAEDA wrote:

Could you please attach screenshots? (before and after). After I see it, I will set target version.

here they are :
  • without patch
  • with patch

#8 Updated by Go MAEDA over 5 years ago

  • Status changed from Needs feedback to Confirmed
  • Target version set to 3.2.2

#9 Updated by Toshi MARUYAMA over 5 years ago

Sorry, I cannot see any difference in note-7 with and without.

#10 Updated by Toshi MARUYAMA over 5 years ago

I got it.

#11 Updated by Jérôme BATAILLE over 5 years ago

Yes it's just a tiny difference.

#12 Updated by Jean-Philippe Lang over 5 years ago

  • Subject changed from % done : progress bar blocked at 80 to % done: progress bar blocked at 80 in the issue list
  • Status changed from Confirmed to Closed
  • Assignee set to Jean-Philippe Lang
  • Target version changed from 3.2.2 to 3.3.0
  • Resolution set to Fixed

Committed, thanks.

#13 Updated by Jérôme BATAILLE over 5 years ago

Thanks a lot.

Also available in: Atom PDF