Github user milamberspace commented on the pull request:

    https://github.com/apache/jmeter/pull/91#issuecomment-175880747
  
    Thanks for your PR.
    Some comments:
    * That would be better to use Constants (final public static)  for 
properties strings inside .java files (like jsyntaxtextarea.font, etc.) 
Particularly with the property key is use into several java files.
    
    * The change inside the file JMeterUtils.java seems not necessary?
    
    * Probably the best place inside the jmeter.properties for this new 
properties is the section Look&feel (start at line ~110)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to