search

monero-project / monero-site

https://getmonero.org
BSD 3-Clause "New" or "Revised" License
275 stars 384 forks source link

user-guides: fix toc for verification guide #1951

Closed selsta closed 1 year ago

selsta commented 2 years ago

Before

Before screenshot

After

After screenshot

Fix by @plowsof

netlify[bot] commented 2 years ago

Deploy Preview for barolo-time-757cf9 ready!

Built without sensitive environment variables

Name Link
Latest commit 2377950d087ca0a5d97d371ca650e6047030017f
Latest deploy log https://app.netlify.com/sites/barolo-time-757cf9/deploys/64038efe90f95500086cbf8a
Deploy Preview https://deploy-preview-1951--barolo-time-757cf9.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.

selsta commented 2 years ago

I did not.

plowsof commented 2 years ago

Good point, i think a custom id tag for this specific fix would be ideal, i will have a look and offer the suggestion

plowsof commented 2 years ago

Here's styling for those specific id's (who decided on double dashes/singledashes ^^), so the changes can be reverted, and these used at the bottom of the file (to save you reviewing) 

h3#--install-gnupg, 
h3#--verify--import-signing-key, 
h3#--download--verify-hash-file, 
h3#--download--verify-binary {
    margin-top: 0rem;
}
selsta commented 1 year ago

@erciccione do I have to update every single file in _i18n/*/resources/user-guides/verification-allos-advanced.md and also these .po files?

erciccione commented 1 year ago

@erciccione do I have to update every single file in _i18n/*/resources/user-guides/verification-allos-advanced.md and also these .po files?

No, only the english file (_i18n/en/resources/user-guides/verification-allos-advanced.md). The po4a script will do the rest (I will run it on all user guides later on)