Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Preserve attribute feature #71

Merged
merged 3 commits into from
Nov 8, 2014

Conversation

roblevintennis
Copy link
Contributor

No description provided.

@roblevintennis
Copy link
Contributor Author

@FWeinb this turned out to be pretty simple to implement. I'm back “in the trenches” for a while but let me know what you think.

@@ -91,7 +91,7 @@
c8.147-1.632,14.055-7.437,14.764-13.808H91V0L34.654,21.67z"/> </symbol><symbol viewBox="0 0 44 39" id="nestedsymbol"><title>menu-pill</title> <symbol viewBox="0 0 44 6" id="nestedsymbol-menu-pill"> <rect fill="currentColor" x="0" y="0" width="44px" height="5px" rx="3px" ry="3px"/></symbol> <text transform="matrix(1 0 0 1 0.1514 38)" fill="currentColor" font-family="&apos;Neue Helvetica eText W01&apos;, sans-serif" font-size="15.5">menu</text> <use xlink:href="#nestedsymbol-menu-pill" width="44" height="6" x="-22" y="-3" transform="matrix(1 0 0 -1 22 21)" overflow="visible"/> <use xlink:href="#nestedsymbol-menu-pill" width="44" height="6" x="-22" y="-3" transform="matrix(1 0 0 -1 22 12)" overflow="visible"/> <use xlink:href="#nestedsymbol-menu-pill" width="44" height="6" x="-22" y="-3" transform="matrix(1 0 0 -1 22 3)" overflow="visible"/> </symbol><symbol viewBox="0 0 100 100" id="pinboard"><title>pinboard</title> <path d="M51.447,59.185c0,0-12.542,12.915-18.583,17.828c-3.561-7.274,2.275-18.725,2.275-18.725l-23.604-30.95
c0,0-7.694,3.517-11.535,2.013C9.471,19.252,19.254,9.47,29.352,0c1.434,3.759-2.012,11.283-2.012,11.283l30.701,23.854
c0,0,11.578-5.927,18.977-2.275c-4.955,5.917-17.83,18.33-17.83,18.33s27.875,31.664,40.66,48.649c0.353,0.301,0.021,0.098-0.252,0
C83.035,86.802,51.447,59.185,51.447,59.185z"/> </symbol><symbol viewBox="0 0 100 100" id="rss"><title>rss</title> <g> <path d="M7.336,62.163c-0.388-0.075-0.783-0.112-1.171-0.112c-1.42,0-2.81,0.491-3.923,1.409C0.822,64.632,0,66.375,0,68.216
C83.035,86.802,51.447,59.185,51.447,59.185z"/> </symbol><symbol id="preserve_attribute"><title>preserve_attribute</title> <rect x="400" y="100" width="400" height="200" preserve--fill="blue" preserve--stroke="red" preserve--imafake="idunno" stroke-width="10"/> </symbol><symbol viewBox="0 0 100 100" id="rss"><title>rss</title> <g> <path d="M7.336,62.163c-0.388-0.075-0.783-0.112-1.171-0.112c-1.42,0-2.81,0.491-3.923,1.409C0.822,64.632,0,66.375,0,68.216
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The preserve-- prefix should be removed here.

…he preserve-- prefix. Also prunes trailing spaces
@roblevintennis
Copy link
Contributor Author

@FWeinb 0faed28 should fix the includedemo preserve-- issue you found. I had neglected to handle the case where they did NOT ask for cleanup..we still need to remove the preserve-- prefix in those cases too.

@FWeinb
Copy link
Owner

FWeinb commented Nov 8, 2014

Great work! Thanks for implementing this. Released in 0.4.1

FWeinb added a commit that referenced this pull request Nov 8, 2014
@FWeinb FWeinb merged commit 9259b50 into FWeinb:master Nov 8, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants