dkandalov / limited-wip

IntelliJ plugin to limit your work-in-progress
https://plugins.jetbrains.com/plugin/7655
111 stars 7 forks source link

Feature Request: support timeout period less than 1 minute #31

Closed EthanLin-TWer closed 4 years ago

EthanLin-TWer commented 4 years ago

Hi there,

Great stuff first of all.

Right now '1 Minute' is the minimum timeout period. Possible for us to set it to like '30 seconds', '15 seconds', '45 seconds'(this three looks sufficient to me)? This is useful when we're doing a Refactor/TDD workshop to enforce people to run tests frequently, or should be something also useful for experienced developer that would like to push themselves to the limits.

image

Feel free to let me know your thoughts, concerns anything.

dkandalov commented 4 years ago

Sounds good to me šŸ‘

dkandalov commented 4 years ago

Not sure if it follows UI guidelines but for now I did this:

image

EthanLin-TWer commented 4 years ago

Well yeap, Iā€™m not sure of the guidelines either but it does serve the purpose.

On Thu, Jan 16, 2020 at 3:45 AM Dmitry Kandalov notifications@github.com wrote:

Not sure if it follows UI guidelines but for now I did this:

[image: image] https://user-images.githubusercontent.com/28375/72465687-8c4f8100-37cf-11ea-9d57-aba782c93453.png

ā€” You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dkandalov/limited-wip/issues/31?email_source=notifications&email_token=AC2YDH5ACPICSNSZTTFENLLQ55RVZA5CNFSM4JZP6IB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJBSK5A#issuecomment-574825844, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC2YDH4YB54SO3ZCS5OAMZ3Q55RVZANCNFSM4JZP6IBQ .

-- Regards,

dkandalov commented 4 years ago

It should be in the plugin version 0.3.10 once it's approved by jetbrains.

EthanLin-TWer commented 4 years ago

As far as I can test out the function works prettier good! Thanks for implementing this quickly!

On Tue, Jan 21, 2020 at 3:55 AM Dmitry Kandalov notifications@github.com wrote:

It should be in the plugin version 0.3.10 once it's approved by jetbrains.

ā€” You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dkandalov/limited-wip/issues/31?email_source=notifications&email_token=AC2YDH3QLYTYAFZDDNSZS23Q6X6SZA5CNFSM4JZP6IB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJNVN5Y#issuecomment-576411383, or unsubscribe https://github.com/notifications/unsubscribe-auth/AC2YDH4KHULQQQTYDIAHZBTQ6X6SZANCNFSM4JZP6IBQ .

-- Regards,