Hi Ariel,<div>A few comments about the patches:</div><div>* Is better have all the changes related to the proposed functionality in a single patch.</div><div><br></div><div>* Do not do changes not related, because is harder understand the patch like in:</div>
<div><br></div><div>                   'front_border': {'fill_color': '#4c4d4f',</div><div>                                    'stroke_color': '#ffffff',</div><div>                                    'opacity': '1'}},</div>
<div>-                None, theme.PAIR_SIZE, 1, '#c0c0c0')</div><div>+                None,</div><div>+                theme.PAIR_SIZE,</div><div>+                1,</div><div>+                '#c0c0c0')</div>
<div><br></div><div>or </div><div><br></div><div><div> </div><div>-    def __init__(self, identifier, pprops, jpeg, size,</div><div>-                 align, bg_color='#000000'):</div><div>+    def __init__(self, identifier, pprops, jpeg, size, align, bg_color='#000000'):</div>
</div><div><br></div><div>* Use a single to save the config, not one for every parameter (now we have two parameters,</div><div>but if we add more is not a good route). Use json is a good idea.</div><div><br></div><div>* Check pep8</div>
<div><br></div><div>* I don't think reading the config from the file in svgcard.py is a good idea.</div><div>Is better use a a method to set it from the other classes using it.</div><div>Having two different methods to read the config is not good.</div>
<div><br></div><div>* Remove the commented code like:</div><div><br></div><div>+                """</div><div>+                f = open("/tmp/cartas.txt", "a")</div><div>+                f.write("ID = " + str(<a href="http://self.id">self.id</a>) + "| TEXT = " + text + "| FONT=" + font + "|" + str(self.pprops) + "\n")</div>
<div>+                f.close()</div><div>+                """</div><div><br></div><div>Gonzalo</div><div><br><div class="gmail_quote">On Fri, Mar 30, 2012 at 3:11 PM, Ariel Calzada <span dir="ltr"><<a href="mailto:ariel@activitycentral.com">ariel@activitycentral.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This patch will let user change font for the cards.<div><br></div><div>Use can edit game and change the font. I attach screenshots</div>
<div><br></div><div>Regards,</div><div>ARIEL CALZADA</div><div><a href="http://www.activitycentral.com" target="_blank">www.activitycentral.com</a><br>
<div><br></div><div><br></div></div>
</blockquote></div><br></div>