I saw the "Top 'page not found' errors" page and I see an error from Menu UI. I saw the CSS is renamed, and the module not use attached for use CSS, so i fixed it.

missing css

I fixed the css too becuse it not had doc comment, and I specified the .menu-enabled style for only the td, because the tr element has too same class.

CommentFileSizeAuthor
#1 2350005_missing_css_1.patch1.97 KBcsakiistvan
Screen Shot 2014-10-03 at 16.54.21.png20.88 KBcsakiistvan

Comments

csakiistvan’s picture

csakiistvan
Primary language Hungarian
Location Budapest, Hungary
commented
Status: Active » Needs review
StatusFileSize
new 2350005_missing_css_1.patch1.97 KB
csakiistvan’s picture

csakiistvan
Primary language Hungarian
Location Budapest, Hungary
commented
Issue summary: View changes
Désiré’s picture

Désiré commented
Status: Needs review » Reviewed & tested by the community

I'm tested it, works well. The code also seems OK.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 1: 2350005_missing_css_1.patch, failed testing.

Status: Needs work » Needs review

Désiré queued 1: 2350005_missing_css_1.patch for re-testing.

Désiré’s picture

Désiré commented

Submitted for retest, since it seems that the test bot were overwhelmed last time.

Status: Needs review » Needs work

The last submitted patch, 1: 2350005_missing_css_1.patch, failed testing.

Status: Needs work » Needs review

Désiré queued 1: 2350005_missing_css_1.patch for re-testing.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Status: Needs review » Needs work
Issue tags: +Needs reroll

Patch no longer applies.

idebr’s picture

idebr commented
Issue tags: +frontend, +CSS, +Regression
idebr’s picture

idebr commented
Status: Needs work » Closed (fixed)
Related issues: +#2361599: menu_ui css file is not loaded

This was fixed in #2361599: menu_ui css file is not loaded

njbarrett’s picture

njbarrett commented
Issue tags: -Needs reroll

No longer required it seems