Opened 8 years ago

Closed 8 years ago

#13883 closed defect (fixed)

Typo in AccordionContainer.less

Reported by: cb1kenobi Owned by: bill
Priority: high Milestone: 1.7
Component: Dijit Version: 1.7.0b1
Keywords: dijit accordion css Cc:
Blocked By: Blocking:

Description

Missing period before "dijitAccordionTitle" on line 98:

.claro .dijitAccordionInnerContainerHover dijitAccordionTitle {
                                          ^

http://bugs.dojotoolkit.org/browser/dojo/dijit/trunk/themes/claro/layout/AccordionContainer.less

Change History (2)

comment:1 Changed 8 years ago by bill

Ah you are right, that's definitely a typo.

There's some redundancy in that file, as earlier it has another rule about hover text:

.claro .dijitAccordionTitleHover {
	color:@hovered-text-color;
}

But the two main rules (with higher precedence) seem to be:

.claro .dijitAccordionInnerContainerSelected .dijitAccordionTitle {
	color:@selected-text-color;
	background-position: 0 0;	/* avoid effect when clicking the title of the open pane */
}

and

.claro .dijitAccordionInnerContainerHover dijitAccordionTitle {
	/* both open and closed */
	color:@hovered-text-color;
}

I'll remove the first rule, and fix the typo in the third.

comment:2 Changed 8 years ago by bill

Resolution: fixed
Status: newclosed

In [26524]:

fix typo and remove redundant rule, fixes #13883 !strict.

Note: See TracTickets for help on using tickets.