search

Access4all / adg

Accessibility Developer Guide
http://www.accessibility-developer-guide.com
Other
183 stars 34 forks source link

feat: replace last update with github link #413

Closed backflip closed 11 months ago

backflip commented 1 year ago

This PR removes the manually maintained Last update date: image

Instead, it displays a new link File history which points to the blame view of the corresponding README.md: image

As the RSS feed still expects a date for every article, we are using the latest Git revision for this. Unfortunately, the feed will show today's date in every entry when we merge this (see https://deploy-preview-413--accessibility-developer-guide.netlify.app/feed/rss.xml). However, I don't see a way around this as removing the changed entry in every README.md will create a new revision.

netlify[bot] commented 1 year ago

Deploy Preview for accessibility-developer-guide ready!

Name Link
Latest commit 913abf3240471b67555e6ae554927779ffc06e39
Latest deploy log https://app.netlify.com/sites/accessibility-developer-guide/deploys/64b8e0bf141acb0008250d6e
Deploy Preview https://deploy-preview-413--accessibility-developer-guide.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 configuration.

christophdubach commented 11 months ago

@backflip: Works as expected (from a nerd 🤣).

I have just two tiny suggestions: I would rename the label from «File history» to «Page history». And I would reduce the .site-contents margin-bottom to 2rem.

backflip commented 11 months ago

@backflip: Works as expected (from a nerd 🤣).

I have just two tiny suggestions: I would rename the label from «File history» to «Page history». And I would reduce the .site-contents margin-bottom to 2rem.

Perfect, fixed.