Issue 9922

Title
Remove tempusdominus dependency
Priority
feature
Status
resolved
Nosy list
ced, pokoli, reviewbot, roundup-bot
Assigned to
ced
Keywords
review

Created on 2020-12-06.23:51:58 by ced, last changed 1 month ago by roundup-bot.

Messages

New changeset 80a91f930bf3 by Cédric Krier in branch 'default':
Convert forgotten format_datetime call to unique format
https://hg.tryton.org/tryton-env/rev/80a91f930bf3
New changeset 9118267a616e by Cédric Krier in branch 'default':
Convert forgotten format_datetime call to unique format
https://hg.tryton.org/sao/rev/9118267a616e
New changeset 8869c5c6fd8f by Cédric Krier in branch 'default':
Remove tempusdominus dependency
https://hg.tryton.org/tryton-env/rev/8869c5c6fd8f
New changeset 238f007f890e by Cédric Krier in branch 'default':
Remove tempusdominus dependency
https://hg.tryton.org/sao/rev/238f007f890e
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2020-12-07.16:33:35

Using this review on firefox it is not possible to create a new attendance line.
The record it's created but all the fields are blank and I can not edit them.

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-12-07.00:16:05
On 2020-12-07 00:08, Sergi Almacellas Abellana wrote:
> I've seen there are two widgets for the same date: One as text and other with the date. 

Clear your cache.
Author: [hidden] (pokoli) Tryton committer Tryton translator
Date: 2020-12-07.00:08:20

I've seen there are two widgets for the same date: One as text and other with the date.
It is possible to only shown one and change the input type with a click like we do for numeric widgets?

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-12-07.00:01:04

Here is review339181002 which removes the usage of tempusdominus.
I used input with type date, datetime-local and time to pop-up a calendar from the native browser support.
The input is used as a button which receive the focus when clicked on the button which is over it.
There is some tricks to ensure that the right place (which trigger the pop-up) of the input is behind the button.
I have tested it on Firefox, Chrome and Edge but also included a test which does not display the button if the browser does not support such input types (like Firefox does not support datetime-local yet). Also on Firefox there is no pop-up for time input, so I do not display the button neither.
There is just one minor issue with Firefox and the trigger of the pop-up. It does not pop directly on the second click on the button, the user has to click a third times.

The review include issue9920 and issue9921.

Author: [hidden] (ced) Tryton committer Tryton translator
Date: 2020-12-06.23:51:58
History
Date User Action Args
2021-02-28 16:23:09roundup-botsetmessages: + msg65017
2021-02-28 16:23:08roundup-botsetmessages: + msg65016
2021-02-21 19:38:50roundup-botsetmessages: + msg64759
2021-02-21 19:38:48roundup-botsetmessages: + msg64758
nosy: + roundup-bot
status: testing -> resolved
2021-02-09 00:36:51reviewbotsetmessages: + msg64406
2021-01-13 00:54:02reviewbotsetmessages: + msg63864
2020-12-07 17:24:51reviewbotsetmessages: + msg62504
2020-12-07 16:33:35pokolisetmessages: + msg62501
2020-12-07 00:16:06cedsetmessages: + msg62495
2020-12-07 00:08:20pokolisetmessages: + msg62494

Showing 10 items. Show all history (warning: this could be VERY long)