Skip to content
This repository has been archived by the owner on Jun 11, 2020. It is now read-only.

Refactoring the UI classes with .ui files #53

Closed
wants to merge 2 commits into from
Closed

Refactoring the UI classes with .ui files #53

wants to merge 2 commits into from

Conversation

FrenchDilettante
Copy link

Hello,

I wanted to work on #12, but it's practically impossible do it in .cpp files...
So, I was wondering if refactoring the dialogs to use qt designer (.ui) files would be something acceptable ? It makes the code cleaner and more readable IMO.
Back to my point, using the designer makes UI integration (using CSS styling) waaaaay easier :)

I started this refactoring on a separate branch, it doesn't seem to be a job too difficult, is it something you would be interested in ? I created this pull request to have your opinion on it.

@nurupo
Copy link
Owner

nurupo commented Sep 18, 2013

I prefer writing UI in code. It's easier to see what properties were changed from their default values and comment on why you did some not so obvious change, which is very important when you try to figure out why X doesn't work, and I find it easier to understand how code works in general, since switching between the UI designer and code is a bit distracting, because of the change of context.
We try to keep the UI code structure simple and organized, settings pages would be the best example of this, so most of the time you shouldn't have trouble visualize what it should look like, but I agree that UI editor is better at visualizing.

So, I was wondering if refactoring the dialogs to use qt designer (.ui) files would be something acceptable ?

Don't think so.

@FrenchDilettante
Copy link
Author

OK, no problem. I'll figure out something else for #12.

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

Successfully merging this pull request may close these issues.

2 participants