-
-
Notifications
You must be signed in to change notification settings - Fork 434
refactor: untangle backend and frontend files #1095
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
Conversation
Computerdores
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good so far (just getting a review in early so as not to do it all at once)
(the global settings should probably not be part of core either)
|
Implemented feedback and just pushed some new changes with some more changes/organizing. Feel free to give feedback on any part of it, some folder structures I'm just feeling out to see how well they work in practice. (Oh man I just saw the number of files this is touching now... at least this seems to be a good time for a refactor like this?) |
Computerdores
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking good so far - though I am curious what do you intend to convey with the widgets/components difference?
As far as the project is concerned probably, though I can't promise anything in terms of reviews since I have my hands full with my thesis |
That one I'm not entirely sure, I believe those folders were pre-existing so I've just been attempting to follow. In my head I've been trying to put parts of widgets that are only ever used for other widgets inside of Components, while widgets that are more "complete" and meant to be used throughout the UI would go in Widgets. Not sure how effective this approach is though, and I'm certainly open to something else. |
No worries, I appreciate any level of feedback I can get! I'll still leave things open for review for a period if they're not urgent on the off-chance, and I'll try not to make any mega-weird changes (outside of #1067 which does fall under the "mega weird" category I feel...) Also congrats and good luck with your thesis! |
fb8f553 to
1dd439d
Compare
|
@Computerdores I'm currently rebasing and removing the I also wanted to note/ask, I just noticed that the MVC folder names were made to be singular, not plural. I'm most used to seeing them pluralized and the rest of the folder structure uses plural names as well. Would you mind if I changed those? Sorry that I'm only realizing this now... |
*goes to cry in the corner*
Feel free to rename them; the singular bled over from another project I was working on at the time where we had all the namespaces as singular. |
For anyone trying to do the same, you need |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(this is just for my sake so I can review the newest commit on its own)
Edit: Nvm Github is trolling second review in a minute
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the idea behing the previews dir?
Having some kind of separation between widgets (anything that can make sense on its own) and components (ones that only make sense as a part of others) might still make sense.
This is because what I envision for the far future (would be interested to know what your thoughts are on that) is basically being able to pick any of the widgets and open an extra window that contains just that or to split an existing window and put different widgets in each pane.
This is how lots of professional software works (Ghidra immediately comes to my mind because I use it somewhat often) and that type of software is what I also associate with the 'Studio' term.
This is where I intend to split out the
I'm certainly open to the idea, I think just at this point in time I'm having difficulties intuiting where some of the existing Qt would files go under that structure, and having that extra outer "widgets" file along with the inner ones wasn't helping with that. Maybe in the meantime,there could at least be the "Components" folders to include Qt files that 100% are sure to be only used as parts of other widgets? |
Ahh okay, I was mostly confused by the
We can also just have it unseparated (for now?). For finding "widgets" specifically looking for subclasses of |
Summary
This is a large refactor of the Qt UI structure with the following intentions:
qt/mixeddirectoryThumbRendererrefactor splitOld Layout
New Layout
Tasks Completed