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

Costing Option Display #161

Closed
wants to merge 1 commit into from
Closed

Costing Option Display #161

wants to merge 1 commit into from

Conversation

hs7898753
Copy link
Contributor

πŸ› οΈ Fixes Issue

Closes #147

πŸ‘¨β€πŸ’» Changes proposed

I suggest showing only the general settings by default and moving the extra settings into a dropdown menu. This will allow users to easily access the settings they need, without being overwhelmed by too much information at once.

πŸ“„ Note to reviewers

πŸ“· Screenshots

Untitled.1.mp4

@MuneebAhmed25211
Copy link

Hi, hope you are doing well.
As far as the issue is concerned, I think there should be 2 different dropdowns, One will be for the general settings in which the first half of the list will be present and in the 2nd dropdown, there will be the extra settings option which will contain the other half. This will clean the settings menu a little bit.
Regards,
Muneeb

@hs7898753
Copy link
Contributor Author

hs7898753 commented Mar 28, 2023

Thankyou @MuneebAhmed25211 for your suggestions.
But IMO having two drop down might also lead to confusion if some settings are present in both dropdowns.

@MuneebAhmed25211
Copy link

Ok @hs7898753 I have another idea If we distribute the settings into different dropdowns under the settings dropdown e.g. I see there are 3 options of length, width and height in the extra settings options. If we combine these 3 into a single dropdown (e.g. dimensions) in the extra settings menu, it will take less space and will be clean. If we choose the names of the dropdowns properly and will make the pairs properly (like length, width, height). I suppose it will make no confusion according to me. Hope you understand.

@hs7898753
Copy link
Contributor Author

Yah! We can do it. thank you

@hs7898753 hs7898753 changed the title Update 2 files Costing Option Display Mar 28, 2023
@nilsnolde
Copy link
Owner

If we combine these 3 into a single dropdown (e.g. dimensions) in the extra settings menu, it will take less space and will be clean.

Yeah I like that idea! It's true, there's so much space wasted actually. Probably we'll need to even get rid of "General" & "Extra" settings and just provide one settings which is unique for the selected mode. Grouping them like that was mostly due to not repeating code that much. But that really made the code much more confusing and spaghetti, maybe it's better to have the settings for each mode cleanly separated in the code as well at the expense of duplicating the same options a few times.

Then it needs a "careful" grouping of costing options which makes sense. The ones you mentioned make sense. There could also be one for e.g. "road classes" which groups use_highways/living_streets/tracks etc. That'd slim down the UI immensely if grouped right.

Good suggestion, thanks!

@hs7898753
Copy link
Contributor Author

hs7898753 commented Mar 31, 2023

Yah! I agree with the suggestion.
@nilsnolde What else I think is how is that if we have a icons for the different grouped settings.
I think it will look good
What you suggest?

@nilsnolde
Copy link
Owner

If there are good icons for what the groups, sure.. I'd say one after the other, and first have a good outline of the grouping

If we go that way, it's a bigger job and more suitable for the GSoC project, rather than now. It fits into the whole re-write we plan to do. I'll transfer the idea to the appropriate issue.

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

Successfully merging this pull request may close these issues.

[BUG] Simplify Costing Options Display
3 participants