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

multiple page tab bars #6387

Closed
steviepubliclab opened this issue Oct 3, 2019 · 10 comments
Closed

multiple page tab bars #6387

steviepubliclab opened this issue Oct 3, 2019 · 10 comments
Labels
bug the issue is regarding one of our programs which faces problems when a certain task is executed

Comments

@steviepubliclab
Copy link
Contributor

I was on this page https://publiclab.org/questions/tag/pm?page=2 scrolling through questions, when I saw it has two scroll bar arrows to toggle between pages:
Screen Shot 2019-10-03 at 10 34 56 AM

@steviepubliclab steviepubliclab added the bug the issue is regarding one of our programs which faces problems when a certain task is executed label Oct 3, 2019
@nstjean
Copy link
Contributor

nstjean commented Oct 3, 2019

I'll take this on. :)

@coderjolly
Copy link
Member

I'll take this on. :)

Sure @nstjean , pls go ahead. Feel free to ask anything. 😄 😄

@nstjean
Copy link
Contributor

nstjean commented Oct 4, 2019

So I've identified the problem - the asked-tab and answered-tab are both showing at all times even though only one is marked active. I can easily update the CSS to hide the inactive one, but there are further problems:

  1. The tabs at the top of the page don't work to switch the active group.
  2. There are no questions in the "answered" section, only in the "asked" section
  3. The paginator for the "answered" section is displaying for the number of items in "asked" - this may be fixed when the logic for 404 page has old menubar #2 is fixed.

My question is, do I solve just the initial problem with a pull request and break up the remainder into new issues? Or should I fix all of it together?

@grvsachdeva
Copy link
Member

If you have a good understanding of related code then go ahead for solving all parts, as you need to increase complexity bit by bit. Thank you!

@nstjean
Copy link
Contributor

nstjean commented Oct 4, 2019

Ok, I'll go ahead and try it! I am new to Ruby but this will be a good challenge.

@grvsachdeva
Copy link
Member

Oh, I hope, it's not your first issue, because you might wanna try an easier issue first.

@nstjean
Copy link
Contributor

nstjean commented Oct 5, 2019

Ok what I'll do is split it up. I'll solve the first part of this, the easy issue, and do a pull request for practice. Then I'll look into what needs to be done for getting the tabs working as a separate issue. :)

@grvsachdeva
Copy link
Member

grvsachdeva commented Oct 5, 2019 via email

@coderjolly
Copy link
Member

So I've identified the problem - the asked-tab and answered-tab are both showing at all times even though only one is marked active. I can easily update the CSS to hide the inactive one, but there are further problems:

  1. The tabs at the top of the page don't work to switch the active group.
  2. There are no questions in the "answered" section, only in the "asked" section
  3. The paginator for the "answered" section is displaying for the number of items in "asked" - this may be fixed when the logic for 404 page has old menubar #2 is fixed.

My question is, do I solve just the initial problem with a pull request and break up the remainder into new issues? Or should I fix all of it together?

Sorry, for replying late but I think @gauravano cleared your doubt.

You can create a PR for the initial problem and the make an issue regarding the remaining !! 😄 😸

@stale stale bot added the stale label Oct 7, 2020
@publiclab publiclab deleted a comment from stale bot Oct 8, 2020
@stale stale bot removed the stale label Oct 8, 2020
@jywarren
Copy link
Member

jywarren commented Dec 1, 2020

Fixed in #8689 ! Thank you!!!!

@jywarren jywarren closed this as completed Dec 1, 2020
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants