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

Date (kit update) #615

Closed
christinaatai opened this issue Feb 17, 2020 · 1 comment · Fixed by #1108
Closed

Date (kit update) #615

christinaatai opened this issue Feb 17, 2020 · 1 comment · Fixed by #1108
Assignees
Labels
new kit Use this when adding a new kit to Playbook (USED IN CHANGELOG)

Comments

@christinaatai
Copy link
Contributor

christinaatai commented Feb 17, 2020

First search through Nitro where date is used to confirm that this change does not affect the UI negatively.

Props:

  • Show day
  • Dark (boolean)
  • Alignment (left, center, right)

Things to note:

  • Month and day should be abbreviated to the first three letters
  • Space on either sides of the dot is 4px
  • Style of dot is body, light
  • Show year if it is not the current year (i.e. if the date is 1/1/2019, show "Jan 1, 2019")

Screen Shot 2020-02-17 at 1 05 21 PM

@christinaatai christinaatai changed the title Date (update) Date (kit update) Feb 17, 2020
@jasoncypret
Copy link
Member

Linked: NUX-638

@jasoncypret jasoncypret added the new kit Use this when adding a new kit to Playbook (USED IN CHANGELOG) label Jun 1, 2020
@andrew-k9 andrew-k9 assigned andrew-k9 and unassigned andrew-k9 Aug 18, 2020
@christinaatai christinaatai linked a pull request Aug 19, 2020 that will close this issue
3 tasks
@coleerikson coleerikson removed a link to a pull request Oct 15, 2020
3 tasks
@coleerikson coleerikson linked a pull request Oct 15, 2020 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new kit Use this when adding a new kit to Playbook (USED IN CHANGELOG)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants