Skip to content

Conversation

@bertrandgorge
Copy link

Hello,

This is a little attempt at adding a new extra property for the description of the page, as computed by the extension Description2.

I was not sure about the unit test so there's some work on that remaining. Appart from that it seems to work as intended.

I let you decide if you think it is worthy adding !

@krabina
Copy link
Contributor

krabina commented Dec 12, 2022

Did you also test what happens if the Description2 extension is not installed?

@bertrandgorge
Copy link
Author

bertrandgorge commented Dec 12, 2022

Hello @krabina yes there's no hard dependency on the Description2 extension as it only looks for the presence of the Parser property "description" (see

if ( $description !== false ) { // set by Description2 extension, install it if you want proper og:description support
)

NB : for the moment I won't personaly be using this code, so there's no pressing at merging anything. It's just there if someone finds it useful enough ?

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.

2 participants