search

xiota / geanylua-scripts

GeanyLua scripts to add some minor features to Geany. Hide the menubar. Show multiple long-line column markers.
GNU General Public License v2.0
3 stars 0 forks source link

auto-sidebar does not respect "Hide sidebar" during tab-browsing #3

Closed ralf3u closed 1 year ago

ralf3u commented 2 years ago

condition

issue a) steps to reproduce 1: put a html-document in focus 2: press shortcut "Hide sidebar" (result: there is no sidebar any more) 3: put a txt-document in focus 4: put back a html-document in focus result: the sidebar is completely empty expectation: there should be no sidebar

possible solution I don't know how to program lua-scripts, but maybe a separate lua-script for the memory could be interesting, to remember if sidebar is disabled.



REMARK if the issue above was solved, it could be that the next two following issues will not appear any more

issue b) steps to reproduce 1: put a html-document in focus 2: press shortcut "Hide sidebar" (result: there is no sidebar any more) 3: put a txt-document in focus 4: put back a html-document in focus result: the sidebar is completely empty, and the shortcut "Hide sidebar" need to be pressed 3x to let appear again the sidebar expectation: there should be no sidebar, and by pressing 1x the shortcut "Hide sidebar", the sidebar should appear

issue c) steps to reproduce 1: put a html-document in focus 2: press shortcut "Hide sidebar" (result: there is no sidebar any more) 3: close Geany 4: reopen Geany result: the sidebar is completely empty expectation: there should be no sidebar, and by pressing 1x the shortcut "Hide sidebar", the sidebar should appear

ralf3u commented 1 year ago

It seems to be that I did a mistake in the title of the report and in the report. The shortcut name is not Hide sidebar but Toggle sidebar.

xiota commented 1 year ago

Fixed in 615c0c4. Requires using lua implementation of "toggle sidebar" to create state file.