You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@royale.apache.org by GitBox <gi...@apache.org> on 2021/02/13 19:46:53 UTC

[GitHub] [royale-asjs] brianraymes opened a new pull request #1079: jewel-daterangerestriction: Replace time calculation with appropriate filters per view state

brianraymes opened a new pull request #1079:
URL: https://github.com/apache/royale-asjs/pull/1079


   The current date range restriction bead is based on time instead of calendar days and doesn't work for each of the date chooser's other views (day, month, and year).
   
   This change disregards time as that doesn't allow the ranges to be inclusive and adds filtering for each view.
   
   The current restriction bead will not allow you to choose the same day for a start and end range. This resolves that issue.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [royale-asjs] carlosrovira commented on pull request #1079: jewel-daterangerestriction: Replace time calculation with appropriate filters per view state

Posted by GitBox <gi...@apache.org>.
carlosrovira commented on pull request #1079:
URL: https://github.com/apache/royale-asjs/pull/1079#issuecomment-778669020


   Thanks for this fix Brian! :)


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [royale-asjs] carlosrovira merged pull request #1079: jewel-daterangerestriction: Replace time calculation with appropriate filters per view state

Posted by GitBox <gi...@apache.org>.
carlosrovira merged pull request #1079:
URL: https://github.com/apache/royale-asjs/pull/1079


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org