Limit open to one instance

Currently it's possible to "stack" open dialogs on top of each other. Would it be a good idea to only allow one open dialog at a time?
Do you happen to have a clever idea to implement this?

[rgieseke]

That would be a great addition!
One way could be to identify the shortcut for „Open file“ and disable/overwrite it in the Textredux dialog. It‘s already found in the hijack function. So maybe it only needs to be re-used.

[rgieseke]

Another way might be to check for a visible Textredux buffer in the current view and re-use this one if a new Textredux buffer is created. This would be a way to go from a "buffer list" to an "open dialog" as well without having to close one Textredux buffer first. (Use-case, realise file not already open, switch to "file open")

Your first suggestion seems to work well enough. There are still some rough edges but it certainly is the right direction.
The other use-case you mention is also interesting, might look into that in the future.

[rgieseke]

Did you just add it manually or did you find a generic way?

Did you just add it manually or did you find a generic way?

Generic but really ghetto. Don't care though since no one except me will use it anyways so it doesn't have to be some super slick solution.

[rgieseke]

I'd care and be happy if you could share it! This would be definitely a good addition to Textredux.

Be my guest. I'll make a feature branch for it soon™.
Just check https://github.com/loomer/textredux/branches in a bit.

As requested the branch exists. Since it's unlikely to get merged in it's current state I'd like to close this. That is unless you see the need for more discussion?

[rgieseke]

I'd like to keep it open because i think it would be a valuable addition - if we find a way that works even if someone doesn't use hijack.

Your version prevents certain keys, maybe the opposite is also possible - explicitly setting permissable keys. Or the other approach could work, re-using Textredux buffers in a view.

I think whitelisting keys would create a much longer list than a blacklist and is IMO not feasible.
The other approach seems more promising but I'm unsure how it could be done. Have you had a chance to think about it?

[rgieseke]

Likely you're right, it could become quite a long list ... (buffer close, quite, switch view, navigate ... on all platforms) -- haven't had a chance to look into filtering on buffer creation ...

[rgieseke]

5d0941e handles this for the open dialog. One can still open e.g. a buffer list. Still not sure how to handle this (there might be other Textredux buffers a user might want to open) but it seems like an improvement.