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

align datetime value to contents in time #411

Closed
wants to merge 1 commit into from

Conversation

himorin
Copy link
Contributor

@himorin himorin commented Jul 29, 2022

Text within time element says 2015年5月22日, but datetime attribute has 2016-05-22. This PR fixes these difference.
These are just test cases or pure example for tate-chu-yoko, so there seems no intention to change between attribute and text.

@himorin himorin requested a review from r12a July 29, 2022 08:08
@netlify
Copy link

netlify bot commented Jul 29, 2022

Deploy Preview for i18n-drafts ready!

Name Link
🔨 Latest commit c9a4ea2
🔍 Latest deploy log https://app.netlify.com/sites/i18n-drafts/deploys/62e39567f7ae4500095f1e80
😎 Deploy Preview https://deploy-preview-411--i18n-drafts.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@r12a
Copy link
Contributor

r12a commented Jul 29, 2022

@himorin many thanks for spotting this.

Actually, when i looked at that article i noticed that many of the items listing browser support were out of date. I have therefore begun applying the latest template to the article (which shows live results in the article itself) and updating the test results. I'll try to integrate these changes you suggest at the same time. I'll let you know when i get further with the conversion.

@r12a
Copy link
Contributor

r12a commented Aug 2, 2022

@himorin i incorporated your changes into 75bda22 so i'll close this now.

@r12a r12a closed this Aug 2, 2022
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.

None yet

2 participants