plots2 icon indicating copy to clipboard operation
plots2 copied to clipboard

Follow button on blog page only works for the first post

Open TildaDares opened this issue 3 years ago • 9 comments

The follow button on the blog posts page only works for the first post.

You can test this by navigating to the blog page https://publiclab.org/blog and clicking the follow button.

The relevant code for this functionality:

https://github.com/publiclab/plots2/blob/bf8b4aba318a8b9c2421271ae44cc733055e1a33/app/views/tag/_subscribe_button.html.erb#L4-L21

https://github.com/publiclab/plots2/blob/bf8b4aba318a8b9c2421271ae44cc733055e1a33/app/views/tag/blog.html.erb#L67

I think the reason it only works for the first post is because the popover id is not unique for each post.

TildaDares avatar Jul 05 '22 07:07 TildaDares

Hi @SteveSayantan, is this something you'd be interested to work on? I'll help with whatever you need.

TildaDares avatar Jul 05 '22 07:07 TildaDares

Hi @TildaDares May I work on this? I just finished my first Contribution here. Also, I think we should use class instead of id for identifying more than one element. The id attribute is intended to be used for a single element and if more than one has the same id, only 1 will be selected. Hence, the first button responds. Or we can use different unique ids but it will be difficult to work with. I'm new please correct me where needed.

udyanojha avatar Jul 10 '22 12:07 udyanojha

Hi @TildaDares May I work on this? I just finished my first Contribution here. Also, I think we should use class instead of id for identifying more than one element. The id attribute is intended to be used for a single element and if more than one has the same id, only 1 will be selected. Hence, the first button responds. Or we can use different unique ids but it will be difficult to work with. I'm new please correct me where needed.

I'll try to reproduce this on my device and then get back!

udyanojha avatar Jul 10 '22 12:07 udyanojha

Hi! @TildaDares , I hope you are doing good. Please refer to the above comments. Thanks

udyanojha avatar Jul 16 '22 18:07 udyanojha

Hi @udyanojha, were you able to reproduce the bug?

TildaDares avatar Jul 16 '22 18:07 TildaDares

Hii @TildaDares can I to fix this issue

Harsh3363 avatar Jul 29 '22 19:07 Harsh3363

Hi @Harsh3363, please go ahead. Thanks!

TildaDares avatar Jul 30 '22 19:07 TildaDares

@Harsh3363 are you still working on this?

hsadia538 avatar Aug 04 '22 14:08 hsadia538

@Harsh3363 are you still working on this?

Yeah

Harsh3363 avatar Aug 04 '22 15:08 Harsh3363

Hi @Harsh3363, do you need help with this?

TildaDares avatar Aug 16 '22 19:08 TildaDares

Hi @Harsh3363, do you need help with this?

Hey @TildaDares Actually my exams are going on and that's why I'm not able to finish this Sorry for the delay

Harsh3363 avatar Aug 17 '22 07:08 Harsh3363

Hi @Harsh3363, that’s perfectly fine.

TildaDares avatar Aug 17 '22 08:08 TildaDares

Completed in #11378

TildaDares avatar Sep 04 '22 15:09 TildaDares