Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Made relativeTime text output title case to match k2 style #1112

Merged
merged 1 commit into from
Mar 28, 2023

Conversation

seejdev
Copy link
Contributor

@seejdev seejdev commented Mar 27, 2023

Made some letters go 🆙

Copy link
Contributor

@RebeccaMahany RebeccaMahany left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm wondering for the future if we'd want to think about handling casing entirely on the launcher side -- we could get into the reverse of this situation easily if the text formatting changes on the k2 side. Maybe something to chat about in the future with @blaedj?

I think usually "a" and "in" wouldn't be capitalized in these strings, but it doesn't look like we have a formal style guide at the moment and this is more visually consistent than it was before, so 🚀 🔥

@directionless directionless merged commit a40c226 into kolide:main Mar 28, 2023
@blaedj
Copy link
Contributor

blaedj commented Mar 28, 2023

Maybe something to chat about in the future with @blaedj?

Yea, we may want to think about this more in the future, but I think right now this is an improvement over what we had!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants