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

UI broken on Linux #7

Open
prokoudine opened this issue Oct 22, 2012 · 6 comments
Open

UI broken on Linux #7

prokoudine opened this issue Oct 22, 2012 · 6 comments

Comments

@prokoudine
Copy link

The UI looks wrong on Linux due to lack of a grid layout on the form.

Here is an example of what I'm talking about and a screenshot of a quick fix for just one problematic area:

GitHub Logo

Would you like a patch for all of UI?

BTW, what about using the slider widget from Kdenlive? You get both a label, a direct numeric input and a slider. Plus everything is perfectly aligned.

GitHub Logo

@Y-Vladimir
Copy link
Owner

Thanks - will think about it for the next SmartDeblur releases!

@prokoudine
Copy link
Author

Wait, but you want the patch for grid layout or not? :)

@Y-Vladimir
Copy link
Owner

If it will look good both for windows and linux version - definitely yes. Could you please show that UI-changes (shown in the picture "After, Ubuntu 12.04") for the windows version?

@prokoudine
Copy link
Author

I simply don't run Windows :) But here is the modified .ui file:

https://dl.dropbox.com/u/55576315/MainWindow.ui

@Y-Vladimir
Copy link
Owner

Thanks!
I have tested it in windows.The main issue - label changes own width during moving slider, as a result I see the slider vibrations during the moving. Second issue - slider positions are fixed in the code (MainWindow::defectTypeChanged method). And it's not so easy to apply proposed changes as is.
But I got the idea with grid layout and will apply it in the next SmartDeblur release.

@programmin1
Copy link

Is this why there's not a listed Linux binary? It's easy enough to compile in Qt Creator on Ubuntu, just make sure libfftw3-dev is installed.

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