Code review comment for lp://staging/~stan/pasaffe/window_title_and_escaping

Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Hi! Thanks for the merge proposal.

ACK on the text escaping.

What's the use case on having the filename in the window title though? I feel that is a detail most users don't need to know when they're using the default database location. Could you perhaps make it only display the filename if it isn't the default file location?

review: Needs Fixing

« Back to merge proposal