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

Refactor project configuration GUI #127

Open
jmkerloch opened this issue Nov 28, 2024 · 6 comments · May be fixed by #133
Open

Refactor project configuration GUI #127

jmkerloch opened this issue Nov 28, 2024 · 6 comments · May be fixed by #133

Comments

@jmkerloch
Copy link
Collaborator

Feature description

For now project configuration is displayed with a table:

image

This was well adapted before the cache management option because there was only 4 columns:

image

We have new feature that must add and this will require adding more configuration for project.

I suggest we change the way project are displayed by using two component in UI:

image

1- A list display all available project and we have button to add / remove / move project
2- When selecting a project, the configuration is displayed

With this approach it will be easier to add more configuration to project:

  • icon ?
  • select multiple location

Additional context

No response

@jmkerloch
Copy link
Collaborator Author

@Guts @xcaeag

I have begun the refactoring:
image

What do you think about this new way of displaying projects ?

With this kind of implementation we can easily add more option for project, or move option for group creation to project.

@xcaeag
Copy link
Collaborator

xcaeag commented Dec 6, 2024

Thanks, the interface looks good.
Please note: today, the “merge with previous” and others options are exclusive. Checkboxes relax this constraint.
We may need to rethink the way these options are configured?

@jmkerloch
Copy link
Collaborator Author

We have a request to add several location : #130

If merge with previous is checked, the other location will be unchecked.

@jmkerloch jmkerloch added the UI label Dec 6, 2024
@jmkerloch jmkerloch linked a pull request Dec 6, 2024 that will close this issue
@Guts
Copy link
Collaborator

Guts commented Dec 6, 2024

It's cool but from my point of view it's not a priority for now. I'll review it anyway.

@jmkerloch
Copy link
Collaborator Author

@Guts I know it's not a priority for now but it will make further development easier from my point of view 😄

@Guts Guts added this to the 2.4.0 milestone Dec 17, 2024
@Guts
Copy link
Collaborator

Guts commented Dec 17, 2024

Some suggestions:

  • add an Apply button to check if the configuration works
  • make the dialog a modal which can be moved without moing the whole QGIS
  • add a checkbox to hide the configuration (warning the end user that it'll be applied to the next QGIS launch)
  • use projects thumbnails instead of table on the left side panel
  • rename options to Global options
  • display cache location and make it easy to open it in explorer

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

Successfully merging a pull request may close this issue.

3 participants