search

FreshRSS / Extensions

A repository containing all the official FreshRSS extensions
GNU Affero General Public License v3.0
322 stars 52 forks source link

Fix non working extension with customised article top bar #60

Closed lapineige closed 4 years ago

lapineige commented 4 years ago

Fix #58

Alkarex commented 4 years ago

@lapineige I have not read the full thread: what DOM elements are you trying to get?

lapineige commented 4 years ago

The article content, without the top and bottom bar around it.

Alkarex commented 4 years ago

.querySelector('.flux_content') should do it then

lapineige commented 4 years ago

I tried, it didn't work. I don't know why.

Alkarex commented 4 years ago

Can you show your code trying .querySelector('.flux_content') ? That class is there especially for that

lapineige commented 4 years ago

Well, I tried again, this time in the real script and not the console… it worked.

marienfressinaud commented 4 years ago

Not sure what I did :thinking: I think that GitHub doesn't like when we work on the master branch of fork repositories. I'll reopen a PR with my own fork

marienfressinaud commented 4 years ago

Should be merged now :)

lapineige commented 4 years ago

Thanks a lot, sorry for not answering.

marienfressinaud commented 4 years ago

No problem, I saw the :rocket: emoji so I considered it was good for you ^^