Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

several UI improvements #11

Closed
pmattern opened this issue Jul 11, 2015 · 2 comments
Closed

several UI improvements #11

pmattern opened this issue Jul 11, 2015 · 2 comments

Comments

@pmattern
Copy link
Contributor

Issue meant to summarize some improvements regarding the UI that I'd deem worth considering but not putting in an issue on its own:

The separator between left pane and the right part of the UI where the actual settings are made cannot be moved. As a consequence those "headers" at the left (Theme, Appearance, ...) are nearly always cut.
Thus I think it would be very good if that separator could e. g. be dragged horizontally.

ObConf-Qt still lacks a preview of themes.
Besides themes are applied at once by clicking I think a preview would be nice to have as it provides an overview of available themes.

Button "Close" can't be activated by a mnemonic (underlined letter that allows for activating by hitting Alt+).
Would it be feasible to implement this?

@PCMan
Copy link
Member

PCMan commented Jul 13, 2015

Preview is hard to do with Qt 5. (easier in Qt4)
Openbox rendering code generates several small X11 pixmaps.
You have to composite them manually and generate a QPixmap, but Qt5 no longer wraps X11 pixmaps with QPixmap. So things becomes much more complicated.
This should be done with some XCB hacks and is not a trivial task.

@pmattern
Copy link
Contributor Author

Considering that Openbox will probably never support Wayland and is hence likely to be dropped as preferred window manager of LXQt I think it shouldn't be put too much effort into its configuration tool ObConf-Qt any more either.
Maybe you can look into the two other items first and postpone or even drop that preview thing. I think the spacer position is annoying but all the rest is far from being essential.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants