Problem/Motivation

When the progress bar first loads, its width is 100%

Proposed resolution

Set the default width to be 3%

Remaining tasks

Write the patch

User interface changes

The progress bar starts at a small width

API changes

None

CommentFileSizeAuthor
#9 progress_bar_starts_at-2486431-9.patch413 bytesLewisNyman
#4 Screenshot 2015-05-11 16.45.34.jpg15.54 KBvinmassaro
#2 progress-bar-starts-at-hundred-2486431-2.patch430 bytesAnonymous (not verified)
Screen Shot 2015-05-10 at 15.22.52.jpg301.83 KBLewisNyman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

LewisNyman’s picture

LewisNyman
he/him
CreditAttribution: LewisNyman at Wunder commented
Issue summary: View changes
Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
FileSize
progress-bar-starts-at-hundred-2486431-2.patch430 bytes

This would be my first patch to core, so bear with me if I did anything wrong. But this one liner fixed it when I tested it.

LewisNyman’s picture

LewisNyman
he/him
CreditAttribution: LewisNyman at Wunder commented
Status: Active » Needs review
vinmassaro’s picture

vinmassaro CreditAttribution: vinmassaro commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
FileSize
Screenshot 2015-05-11 16.45.34.jpg15.54 KB

Nice work, this fixes the issue for me. Setting to RTBC.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

This has bugged in me for ages. Thanks! Committed 79e74a2 and pushed to 8.0.x. Thanks!

  • alexpott committed 79e74a2 on 8.0.x 
    Issue #2486431 by vilepickle, vinmassaro: Progress bar starts at 100%

Status: Fixed » Closed (fixed)

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

LewisNyman’s picture

LewisNyman
he/him
CreditAttribution: LewisNyman at Wunder commented
Status: Closed (fixed) » Needs work

This issue was accidentally reverted by #2395853: Split system.module.css and system.theme.css files into SMACSS style components.

LewisNyman’s picture

LewisNyman
he/him
CreditAttribution: LewisNyman at Wunder commented
Status: Needs work » Needs review
FileSize
progress_bar_starts_at-2486431-9.patch413 bytes

New patch

Manuel Garcia’s picture

Manuel Garcia CreditAttribution: Manuel Garcia at Appnovation commented
Status: Needs review » Reviewed & tested by the community

Looks good to me, also, good catch!

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott at Chapter Three commented
Status: Reviewed & tested by the community » Fixed

Committed 70ecd27 and pushed to 8.0.x. Thanks!

  • alexpott committed 70ecd27 on 8.0.x 
    Issue #2486431 followup by LewisNyman: Progress bar starts at 100%

Status: Fixed » Needs work

The last submitted patch, 9: progress_bar_starts_at-2486431-9.patch, failed testing.

LewisNyman’s picture

LewisNyman
he/him
CreditAttribution: LewisNyman at Wunder commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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