Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

--counter: accept colon character (useful for time-tracking apps with hour:min) #1378

Merged
merged 2 commits into from
Apr 11, 2022

Conversation

trkoch
Copy link
Contributor

@trkoch trkoch commented Apr 4, 2022

Modify regexp to track hour counters such as (1:23).

Modify regexp to track hour counters such as `(1:23)`.
@trkoch
Copy link
Contributor Author

trkoch commented Apr 4, 2022

I noticed app/src/helpers/helpers.test.ts is not included in the test files (ignored with testPathIgnorePatterns in package.json). Therefore, I couldn't verify the test I've added.

@ronjouch
Copy link
Contributor

ronjouch commented Apr 10, 2022

@trkoch context/screenshot, please. How does this end up displaying in your dock? It supports and correctly displays the colon?

@trkoch
Copy link
Contributor Author

trkoch commented Apr 11, 2022

It supports and correctly displays the colon?

Yes.

image

@ronjouch ronjouch changed the title Support hour counter with colon --counter: accept colon character (useful for time-tracking apps with hour:min) Apr 11, 2022
@ronjouch ronjouch merged commit 6a949ca into nativefier:master Apr 11, 2022
@ronjouch
Copy link
Contributor

@trkoch merged, thanks! Will ship in the next release, in a few days/weeks.

Adam777Z pushed a commit to Adam777Z/nativefier that referenced this pull request Nov 9, 2022
… hour:min in title (PR nativefier#1378)

Modify regexp to track hour counters such as `(1:23)`.

This is supported by macOS' dock and displays in the red badge:
https://user-images.githubusercontent.com/73974/162786478-609a90e1-5efb-44ba-9aa5-7a3038f0689b.png
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants