@abmorris > > True, but removing them is on the OPs bugs checklist at the beginning of > > the PR, and then it will not be backward compatible.
> It is less work for me not to implement this, if you prefer. > It leaves the file a bit untidy, but it should be harmless. Ok, sessions won't share between versions, but at least old versions won't get their sessions deleted, so no option is ok. ``` [geany] msgwindow_position treeview_position geometry ``` Yes, geometry relates to the configuration of the machine, moving from a system with two screens to one with one screen or large screen to small screen can result in Geany being offscreen, so its a setting that should stay with the system, ie session (thats why the options to not save it IIRC). Also search dialog location `[search] position_*` is the same, its a separate window and might be offscreen, we have had issues where people unplugged their second monitor and lost it. It would more than likely be useful if those were moved to session, but its up to you, if all your sharing machines are identical you won't need it. Could also be a separate PR to keep this small. The internal window positions are less important, your choice if you do them now. > A "clear history" button (as suggested in #1763 (comment)) which checks > geany.conf and (deletes?) session.conf would take care of this, but I see > that as a separate feature for another PR. Yes, if someone needs it, they can make a PR later @kugel- > Yea, lets concentrate on the session and project file lists for this PR. +1 someone doing a separate PR for projects when we get our ducks in a row is fine, but it never hurts to raise it So lets not pollute this PR with that discussion any more and we can continue it on another issue/PR when (if) it gets created. IIRC there have been undecided discussions before raising many of the same things, that we probably need to revisit, I vaguely remember some incredibly complex suggestion I had at one point :grin:. As @abmorris said, this is stand alone, and I will add smaller PRs are more likely to get merged quickly. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany/pull/2776#issuecomment-813755065
