View Single Post
Old 06-21-2011, 02:26 AM   #123
ldolse
Wizard
ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.ldolse is an accomplished Snipe hunter.
 
Posts: 1,337
Karma: 123455
Join Date: Apr 2009
Location: Malaysia
Device: PRS-650, iPhone
Quote:
Originally Posted by Agama View Post
How about simply removing the entire @page declaration and replacing it with a new one defined by the user, (in the plugin). This removes the need for any clever parsing.
The theme of Kiwidude's plugin is to do minimal damage/modification, which I fully agree with. @page could potentially specify things other than margin, and I wouldn't want to delete such things with a wholesale replacement. Aside from that replacing an existing @page declaration is different logic from inserting one that doesn't exist, the way the function works at the moment is it only modifies things which already exist (and removes them from the css if they're empty after removing margins).
ldolse is offline   Reply With Quote