[#MDL-17218] broken treeview styles in YUI 2.6 - Moodle Tracker

Details

Type: Bug
Status: Closed
Priority: Minor
Resolution: Fixed
Affects Version/s: 1.9.3
Fix Version/s: 1.9.4, 2.0
Component/s: Libraries
Labels:
None

Database:

Any
Affected Branches:
MOODLE_19_STABLE
Fixed Branches:
MOODLE_19_STABLE, MOODLE_20_STABLE

Description

Moodle 1.9 updated to YUI 2.6. However, it seems like they accidentally misplaced one of their sprite files in YUI 2.6, so the treeview menu doesn't display right. I don't quite know what happened with the file, since they don't have their VCS publicly available all I know is that it was there in 2.5.2 and not in 2.6.

This is affecting the contrib block yui_menu and may affect any thing else that uses treeview. I submitted a bug report a while back (http://sourceforge.net/tracker2/?func=detail&aid=2173222&group_id=165715&atid=836476) but it has not yet been fixed.

More Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

yui_menu_0.3.jpg

22 kB

14/Nov/08 10:15 AM

Activity

Ascending order - Click to sort in descending order

Hide

Permalink

Ashley Holman added a comment - 14/Nov/08 10:15 AM

Hi Allan,

Can you please tell me which file I need to add back in from 2.5? I'm looking to fix this asap on some of my installations here - just upgraded to 1.9.3 and the course menus don't display correctly (screenshot attached).

Cheers

Show

Ashley Holman added a comment - 14/Nov/08 10:15 AM Hi Allan, Can you please tell me which file I need to add back in from 2.5? I'm looking to fix this asap on some of my installations here - just upgraded to 1.9.3 and the course menus don't display correctly (screenshot attached). Cheers

Hide

Permalink

Alan Trick added a comment - 15/Nov/08 2:30 AM

Replace `treeview.css`, I believe (under lib/yui/treeview/assets/). You should be able to add that with no ill side effects. I've just replaced the whole thing with the previous version of YUI and that worked too.

Show

Alan Trick added a comment - 15/Nov/08 2:30 AM Replace `treeview.css`, I believe (under lib/yui/treeview/assets/). You should be able to add that with no ill side effects. I've just replaced the whole thing with the previous version of YUI and that worked too.

Hide

Permalink

Eloy Lafuente (stronk7) added a comment - 17/Nov/08 7:04 PM

Assigning to Nicolas... as he knows more about the last YUI update...

Show

Eloy Lafuente (stronk7) added a comment - 17/Nov/08 7:04 PM Assigning to Nicolas... as he knows more about the last YUI update...

Hide

Permalink

Nicolas Connault added a comment - 20/Nov/08 11:38 PM

Copied the image where the css expects it to be. I left the original one where it was to avoid any other dependency issues (lib/yui/treeview/assets/skins/sam).

Show

Nicolas Connault added a comment - 20/Nov/08 11:38 PM Copied the image where the css expects it to be. I left the original one where it was to avoid any other dependency issues (lib/yui/treeview/assets/skins/sam).

Hide

Permalink

Nadav Kavalerchik added a comment - 13/Aug/09 11:21 PM

related issue (workaround for rtl icons too)
http://tracker.moodle.org/browse/MDL-19769

Show

Nadav Kavalerchik added a comment - 13/Aug/09 11:21 PM related issue (workaround for rtl icons too) http://tracker.moodle.org/browse/MDL-19769

Hide

Permalink

Rossiani Wijaya added a comment - 22/Sep/09 10:24 AM

Resolved.

Closing

Show

Rossiani Wijaya added a comment - 22/Sep/09 10:24 AM Resolved. Closing

People

Assignee:

Nicolas Connault

Reporter:

Alan Trick

Tester:

Rossiani Wijaya

Participants:

Alan Trick, Ashley Holman, Eloy Lafuente (stronk7), Nadav Kavalerchik, Nicolas Connault, Rossiani Wijaya

Vote (0)

Watch (2)

Dates

Created:

13/Nov/08 2:06 AM

Updated:

22/Sep/09 10:24 AM

Resolved:

20/Nov/08 11:38 PM