Skip to content
This repository has been archived by the owner on Sep 1, 2020. It is now read-only.

Small fixes #28

Merged
merged 8 commits into from Nov 12, 2012
Merged

Small fixes #28

merged 8 commits into from Nov 12, 2012

Conversation

Aerilius
Copy link
Contributor

This contains small fixes for #14,#24, #25, #27 and some others. The "suggestions" were only meant optional (for discussion) but I seem to be unable to deselect them.

• add the input to the console content
• turn the result into a string, so it can be added to the console content
• `elem.getAttribute('class')` is null in IE; replaced by 
  `elem.getAttribute('class') || elem.className`
• `elem.removeAttribute('class')`   replaced by `su.removeClass()`
• `elem.className=`   replaced by `su.addClass()`
• new functions added: `su.toggleClass()`, `su.hasClass()`
scottlininger pushed a commit that referenced this pull request Nov 12, 2012
@scottlininger scottlininger merged commit 3a60ba0 into SketchUp:master Nov 12, 2012
@scottlininger
Copy link
Contributor

Looks good. Merged.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants