Skip to content

Add more advanced options to ui#1212

Open
DIDONAX wants to merge 11 commits intomotis-project:masterfrom
DIDONAX:advanced-options-to-ui
Open

Add more advanced options to ui#1212
DIDONAX wants to merge 11 commits intomotis-project:masterfrom
DIDONAX:advanced-options-to-ui

Conversation

@DIDONAX
Copy link
Copy Markdown
Contributor

@DIDONAX DIDONAX commented Dec 20, 2025

fixes #1015
added:

  • transferTimeFactor
  • additionalTransferTime
  • cyclingSpeed
  • pedestrianSpeed

Comment thread ui/src/lib/AdvancedOptions.svelte
Comment thread ui/src/lib/AdvancedOptions.svelte
Comment thread ui/src/lib/i18n/de.ts Outdated
Comment thread ui/src/lib/i18n/fr.ts Outdated
@DIDONAX DIDONAX force-pushed the advanced-options-to-ui branch from 8567839 to f279fed Compare December 29, 2025 16:50
Comment thread ui/src/lib/AdvancedOptions.svelte Outdated
Copy link
Copy Markdown
Contributor

@traines-source traines-source left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

I think grid alignment should be improved.
Also, I think that if we display the speeds, they should be converted to km/h, because at least I don't really have an intuitive understanding of m/s.
Also not sure why the bike speed slider does not distinguish itself visually if it's disabled, that can be confusing I guess...

@DIDONAX
Copy link
Copy Markdown
Contributor Author

DIDONAX commented Dec 30, 2025

maybe for cycling speed, but for pedestrian i find m/s more intuitive because i can approximate my speed on the fly assuming 1step ~ 1m, and same goes for wheelchair i am sure they have similar intuition. I would say for running/jogging km/h is better but the speed limit doesn't reach that unless i increase it, which i think i should.

@traines-source
Copy link
Copy Markdown
Contributor

I hadn't thought about jogging before, you're right that this could be a use case at least for direct itineraries. So yeah why not increase the limit. I would still favor km/h. E.g. 1.2 m/s sounds really fast to me but 4.32 km/h is actually quite slow (for myself I'm always calculating with 6km/h - 10min/km (which would be yet another unit that one could use :D). But yeah maybe not worth the effort with the stepping etc. so I'm open to leave it with m/s for the time being.

@DIDONAX DIDONAX force-pushed the advanced-options-to-ui branch from 082fc9b to 50f88c7 Compare January 3, 2026 03:49
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.

Add all advanced options from API to UI

2 participants