search

bhadaway / blankslate

A naked WordPress theme, powering over 10k websites. Buy me a sandwich. 🥪
https://opencollective.com/blankslate
328 stars 92 forks source link

Make entry-summary time datetime format W3C-valid #35

Closed robertandrews closed 2 years ago

robertandrews commented 2 years ago

Re: #32 --

If I understand correctly, get_the_date() leverages the date format set on options-general.php. But this is likely intended to output a human-readable date.

Where used as the datetime property in the time element, I think this should be enforced as a W3C-compliant machine date format.

Human-readable format should be left in tact as the element content.

I'm pretty new at this. This is my first attempt at committing a change to a forked repo, and first pull request. Please forgive/feed back on any faux pas.

robertandrews commented 2 years ago

Apologies if the p-in-span stuff is popping up in here. I'm trying to do a separate pull request for that, don't know why it seems to be attaching to this. Newbie.

robertandrews commented 2 years ago

... I guess, to file changes against each individual issue raised, I would need to create and commit a new branch? Is that why my subsequent commits are attaching to this same one?