Skip to content

Set new time for "endTime" does not work; BUG #22

Open

Description

@salierykuba

timeRangePicker.startTime = TimeRangePicker.Time(12, 0) : work ok

timeRangePicker.endTime = TimeRangePicker.Time(12, 0) : does not work, does nothing

Activity

  1. tomleejumah commented on Feb 16, 2024

    @tomleejumah

    Actually the bug or rather the design is that you cant use EndTime or StartTime together since they all reference to same thumb inside onCreate but when you slide around it works for both thumbs...I wish It can be fixed.

  2. Droppers commented on Feb 16, 2024

    @Droppers
    Owner

    I don't understand the issue, can you elaborate, or point to the incorrect code location? Thank you :)

  3. tomleejumah commented on Feb 16, 2024

    @tomleejumah

    I don't understand the issue, can you elaborate, or point to the incorrect code location? Thank you :)

    here is a link to myGist I have highlighted the "bug" https://gist.github.com/Ttoml33/0640f5950faca65014983ab17c76ed2e

  4. salierykuba commented on Feb 17, 2024

    @salierykuba
    Author

    Actually the bug or rather the design is that you cant use EndTime or StartTime together since they all reference to same thumb inside onCreate but when you slide around it works for both thumbs...I wish It can be fixed.

    thank you; That's exactly the problem; For example; When starting the activity I want to be able to set the endTime and startTime at the time I decide; but it doesn't work, as you explain.
    The only thing I could do to fix it was to set the endTime using: timeRangePicker.endTime.totalMinutes. It works, but it is not correct.
    I also hope they fix it soon

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions