search

JasonLautzenheiser / trizbort

Trizbort, the Interactive Fiction Mapper
http://www.trizbort.com
Other
130 stars 35 forks source link

Solutions for #387, #452, #454, #460 #462

Closed Nyun0 closed 5 years ago

Nyun0 commented 5 years ago

Solution propositions for various minor issues.

What is worth consideration:

  1. Recent Files list length could be configurable in the app settings, instead of having a hardcoded 4
  2. Adding Paint/Repaint methods in the Room class
  3. There is one more place where ApplicationSettingsController.AppSettings.RecentProjects is using .Add instead of .Insert
JasonLautzenheiser commented 5 years ago

These all appear fine and I will merge them in.....

In the future, it would be easier if each of these issues were separated into separate pull requests. When they are mixed into one like this, it makes it harder to test each one individually.

Thanks