Support from Acquia helps fund testing for Drupal Acquia logo

Comments

csakiistvan’s picture

I clicked the "Check Manually" link, but i see the module list.
I think bigger problem that i don't see the progress bar in move. It's background color is white.

markhalliwell’s picture

Title: Update manager page is blank when checking for updates... » Progress bar not working
Version: 7.x-3.0-rc1 » 7.x-3.x-dev
Issue tags: +7.x-3.0-rc2
csakiistvan’s picture

Class changes:
Bootstrap 2 class: .progress-info.progress-striped .bar https://dl.dropboxusercontent.com/u/743951/Bootstrap_2.png
Bootstrap 3 class: .progress and .progress-bar https://dl.dropboxusercontent.com/u/743951/Bootstrap_3.png

http://getbootstrap.com/components/#progress

valkum’s picture

Here is a patch.
changed js and non_js progress bars to bootstrap html structure.

valkum’s picture

Status: Active » Needs review
markhalliwell’s picture

Status: Needs review » Needs work
  1. +++ b/theme/alter.inc
    @@ -183,6 +183,7 @@ function bootstrap_js_alter(&$js) {
    +
    

    Unnecessary whitespace addition.

  2. +++ b/theme/system/progress-bar.tpl.php
    @@ -0,0 +1,19 @@
    + * progress.tpl.php
    

    needs to be progress-bar.tpl.php

valkum’s picture

Status: Needs work » Needs review
FileSize
9.57 KB

Oh sry missed that.

markhalliwell’s picture

Status: Needs review » Needs work

#6.1 still exists.

valkum’s picture

removed the lines from the wrong patch file.
reupload.

valkum’s picture

Status: Needs work » Needs review
markhalliwell’s picture

Status: Needs review » Fixed

Thanks @valkum!

Committed a2c72c6 to 7.x-3.x.
Committed ced34fd to 7.x-3.x.

valkum’s picture

Version: 7.x-3.x-dev » 7.x-3.0-rc2

Automatically closed -- issue fixed for 2 weeks with no activity.