calendar overlay widget style broken after lazr-js 191 upgrade
Bug #684911 reported by
Deryck Hodge
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Critical
|
Huw Wilkins | ||
Bug Description
The calendar overlay widget used on many registry pages (for example, https:/
This bug is meant to track the need for the fix and the fact that it's a regression after the upgrade.
Related branches
lp:~huwshimi/launchpad/broken-calendar-684911
- Curtis Hovey (community): Approve (code ui)
-
Diff: 11 lines (+2/-2)1 file modifiedlib/canonical/launchpad/icing/yui_2.7.0b/build/calendar/assets/skins/sam/calendar.css (+2/-2)
| affects: | launchpad-registry → launchpad-web |
| Changed in launchpad-web: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| tags: | added: lazr-js-upgrade |
| Changed in launchpad: | |
| importance: | High → Critical |
Revision history for this message
| Huw Wilkins (huwshimi) wrote | #1 |
Revision history for this message
| Deryck Hodge (deryck) wrote | #2 |
| Changed in launchpad: | |
| milestone: | none → 11.02 |
| assignee: | nobody → Huw Wilkins (huwshimi) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #3 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Just had a quick look and the reason it has no styles is because the YUI skin is not being applied. Applying the skin manually (i.e. adding the appropriate class name) fixes it.
I can't see anything that handles applying the skin. I assume this was happening somewhere previously if an upgrade broke it.