Nice work!
But instead of shaming you for modifying core files, I'd better try draw Gideon's (the author of Q2A) attention to the BIG problem - it is impossible to make a really good custom theme for Q2A without modifying core files!
There are a lot of HTML rendering being done in the core and you can't change it without hacking inside. I'd like to make a proposition, that ALL HTML should be rendered as a part of theme. Basic HTML rendering, that is being thought as common for all themes and is rendered within a core now, should be refactored as ViewHelpers (or at the simpliest case as a part of qa_theme_base class), that can be easily overridden by theme creator.