* It looks either a bit confusing or to have some bugs (without having investigated deeply, it did things I didn't understand); * It marks all lines when reloading a file (instead of resetting to a blank slate); * If marker margin is not visible, it can render the editor unreadable with the default settings; * I don't see any obvious use case given its current behavior that warrants a so visible feature by default; * I don't think the feature is obvious at first glance, which could easily lead users to wonder what that line is about. It also looks similar to other editor's VCS commit status indicator (including git-changebar Geany plugin) display, yet behaves very differently, possibly increasing confusion.
So for now, I think it's better to have it disabled by default. See also discussion starting at https://github.com/geany/geany/pull/3551#issuecomment-1751809855 Opinions @kugel- @eht16 @techee @elextr @ntrel and everyone else? You can view, comment on, or merge this pull request online at: https://github.com/geany/geany/pull/3591 -- Commit Summary -- * Do not enable Scintilla's Change History by default -- File Changes -- M src/keyfile.c (2) -- Patch Links -- https://github.com/geany/geany/pull/3591.patch https://github.com/geany/geany/pull/3591.diff -- Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/3591 You are receiving this because you are subscribed to this thread. Message ID: <geany/geany/pull/3...@github.com>
