Website menu on mobile

[amueller]

The website menu doesn't look great on mobile. We could probably improve.
This is a good issue for someone who wants to contribute but doesn't have a lot of machine learning experience.

[mblondel]

Digression: what's the point of the "need contributor" tag? Pretty much every issue needs a contributor ;)

[nelson-liu]

I suppose it's mainly for newcomers who want to contribute to the project. I noticed at the sprint that lots of people were struggling to find things to work on, which I feel could be mitigated if we were better about applying appropriate labels.

[mblondel]

Indeed but there is also the "easy" label for that.

[vharavu]

Re: the digression, if an Open issue has a PR contributed by someone, but the PR isn't yet merged, does one not work on the issue? How do I find an easy issue to work on that someone else hasnt already made a PR?
thank you

[nelson-liu]

@vharavu generally, you look at whether the reason the PR isn't merged is inactivity by the reviewers or inactivity by the submitter. If the submitter hasn't responded to the last round of reviews in awhile, it's reasonable to comment on the PR asking the submitter if they are still working on it / whether you can take over. If the reviewers haven't gotten back (likely case), it's generally best to not work on it.

Ideally, to find an easy issue to work on that someone hasn't made a PR to, you'd just filter by "Need Contributor" and "Easy" ;). However, lots of issues that are easy and need a contributor are not tagged as such (hence my response to the digression). Untouched issues are generally on the first page of github; else you unfortunately have to look at every issue you're interested in and see if there's someone working on it (this is my experience at least, if someone has a better way please share! :) )

[Ajackster]

@amueller I'm interested in working on this. I am pretty crafty with front end web development and I'm new to machine learning. How would you like to coordinate the formatting of the website menu on mobile?

[amueller]

@mblondel most issues don't actually need someone to jump in, they are already addressed by a PR, which needs either review or is waiting for the author to address the review. The idea is that once someone starts on an issue, the tag is removed.

[amueller]

@Ajackster Great, pleas just go ahead with what you think is reasonable. We don't really look for an entire redesign. We are using bootstrap for the website, so in principle it shouldn't be that hard.
Open a PR, reference this issue and attach a screenshot.

Mostly I don't like how the menu wraps.

[tguillemot]

@amueller Can we close this issue ?