Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"# more" link not working in tags sidebar listing on nodes pages #8490

Closed
ebarry opened this issue Oct 8, 2020 · 10 comments
Closed

"# more" link not working in tags sidebar listing on nodes pages #8490

ebarry opened this issue Oct 8, 2020 · 10 comments
Labels
bug the issue is regarding one of our programs which faces problems when a certain task is executed fto-candidate issues which are meant to be solved by first timers but aren't well-formatted yet

Comments

@ebarry
Copy link
Member

ebarry commented Oct 8, 2020

Please describe the problem

The underlined link text in the sidebar that indicates that there are a certain number of additional tags on a piece of content is clickable, but is not actually a link
Screen Shot 2020-10-08 at 1 13 08 PM

What did you expect to see that you didn't?

I expected that this would expand to show existing tags.
Using the "circle +" does serve to expand and show the tags AND provide the add a tag box, so the functionality is still available, which is good.

However, the broken link should be fixed.

Please show us where to look

https://publiclab.org/questions/jeffkrol/09-29-2020/using-thunderoptics-spectrophotometer

What's your PublicLab.org username?

This can help us diagnose the issue:
liz

@ebarry ebarry added bug the issue is regarding one of our programs which faces problems when a certain task is executed fto-candidate issues which are meant to be solved by first timers but aren't well-formatted yet labels Oct 8, 2020
@swarajpure
Copy link

Can I work on this?

@cesswairimu
Copy link
Collaborator

Go ahead @swarajpure. Thanks.
The file that renders this tag sidebar is https://github.com/publiclab/plots2/blob/main/app/views/tag/_tagging.html.erb

@ebarry
Copy link
Member Author

ebarry commented Oct 8, 2020

Thanks @swarajpure !
Feel free to ask any questions, i may not have described the issue very well.
Clicking the underlined text is supposed to show the blue lozenges of the tags. Then clicking the "circle+" is supposed to open the input box to add more tags. This is right, isn't it @jywarren ?

@swarajpure
Copy link

Thank you @ebarry !
Do I necessarily have to have linux environment for ruby? I use windows.

@cesswairimu
Copy link
Collaborator

cesswairimu commented Oct 9, 2020

@swarajpure nope, we do have windows installation guidelines here https://github.com/publiclab/plots2#windows-installation do let us know if you are having any trouble with the installations. Thanks

@swarajpure
Copy link

I guess I will pass this one. Commenting myself so that somebody else can take it up and fix it instead of you guys waiting up for me. Didn't wish to ghost.

Sorry for the inconvenience caused.

@cesswairimu
Copy link
Collaborator

no worries, thanks @swarajpure

@jywarren jywarren changed the title Link not working "# more" link not working in tags sidebar listing on nodes pages Oct 13, 2020
@jywarren
Copy link
Member

Hi all! this could make a pretty good first-timers-only issue -- the code for the link itself is here:

<% if i == 2 && !power_tag %>
<a class="show-more-tags" href="javascript:void(0);"><p style="float:left; color:#666; margin-top:14px; margin-left:5px;"><u><%= tags.length - 2 %> more</u> &nbsp </p></a>
<% end %>

And, the JavaScript which opens the extra tags was modified recently, so we just need to change the selector

$("#tags-open").click(function(){
$("#tagform").show();
$("#tags-open").hide();
$('.more-tags').show();
$('.show-more-tags').hide();
$(".tag-input").focus();
});

to include the text link too, so I think it would be:

   $("#tags-open, .show-more-tags").click(function(){ 

This has been marked as a good candidate for becoming a first-timers-only issue like these, meaning that it's simple, self-contained, and with some extra formatting, could be a great entry point for a new contributor. If you're familiar enough with this code, please consider reformatting or reposting it as a first-timers-only issue, and then ping @publiclab/reviewers to get it labelled. Or, if this is not your first time, try to solve it yourself!

@publiclab-mimi
Copy link
Collaborator

Hi all. I wanted to note that this broken link is especially troublesome if you’re not logged in to the website, as I’m only able to see the functional “+” button if I’m logged into Public Lab. So for food who may not have a username, they’re unable to access any tags that are in the collapsed menu.

@jywarren
Copy link
Member

Hi, just noting this was solved in #8571 by @nk183 in a first-timers-only issue created by @Cadreia -- thank you all!!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug the issue is regarding one of our programs which faces problems when a certain task is executed fto-candidate issues which are meant to be solved by first timers but aren't well-formatted yet
Projects
None yet
Development

No branches or pull requests

5 participants