tkTimePicker icon indicating copy to clipboard operation
tkTimePicker copied to clipboard

Added seconds functionality fix some minor bugs

Open dyllbert opened this issue 1 year ago • 2 comments
trafficstars

One feature addition and some other bug fixes

Features

Added seconds

  • Second options now exist for all three timepicker interfaces.
  • Added needed functions and classes.
  • Modified functions that involve all aspects of the time picker (such as addAll() or time()) to now also involve the seconds.
  • This included making changes to the separators to distinguish between the HH : MM and MM : SS separators.

Bugs

Separators

  • The separators will now be added automatically when manually adding components to the spin timepickers.
  • This includes both the hour/min and min/second separator.

Time function hour reporting

  • If the the hour was setup using the addHours12() or addHours24() functions, the time() function would not properly report the time.
  • This was fixed by setting the self.hour_type value to the correct constant in the addHours12() or addHours24() functions

dyllbert avatar Jun 20 '24 16:06 dyllbert

Hi, Thanks for creating this pull request, I am a bit busy this month, I'll come back and look into this when ever I get time.

PaulleDemon avatar Jun 26 '24 01:06 PaulleDemon

No worries! I did it for myself and have the changes locally where I need them, but thought it could be useful to others. Cheers!

On Tue, Jun 25, 2024, 19:34 PaulleDemon @.***> wrote:

Hi, Thanks for creating this pull request, I am a bit busy this month, I'll come back and look into this when ever I get time.

— Reply to this email directly, view it on GitHub https://github.com/PaulleDemon/tkTimePicker/pull/10#issuecomment-2190355878, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIUKU7R6GMFSIFR53ZEGEXLZJILC5AVCNFSM6AAAAABJULBHG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJQGM2TKOBXHA . You are receiving this because you authored the thread.Message ID: @.***>

dyllbert avatar Jun 26 '24 02:06 dyllbert