[PATCH 00/46] A re-styling of Patchwork

[Damien Lespiau]

On Sun, Nov 23, 2014 at 05:57:23PM +0100, Wolfram Sang wrote:
> 
> > >   http://patchwork.lespiau.name/
> > 
> > Wow, looks great!
> 
> It has pros and cons. Can the font size in the patch display list be
> configured? Current patchwork fits more on a page and I like that.

The number of the lines one can see on a single page is an interesting
point. Two things can be done there, use a sighlty smaller font and
reduce the vertical padding a bit. Will give it a try.

> I filter my patch list a lot by partial submitter names. The new UI
> doesn't allow that, it seems.

I'm wondering what's the problem here. The auto-completion should
present you with the relevant list of submitters. Are you saying there's
a valid case for searching for "Thom" and get all the Thomas? That with
the way it was done before it was fewer keys to type?

> While I am usually a fan of zebra stripes, highlighting the patch which
> the mouse cursor is over is really a good thing.

Yey!

> Maybe you can also fix something which annoys me for quite some time
> now, but I couldn't fix it. When using the login-page, place the cursor
> immediately on the username field?

If you want to give it a try, using jquery, that should be something
like:

<script>
  $(function() {
      $('#id_username').focus()
  });
</script>

ie, on the document load event (DOM ready!), find the element with the
'id_username' id and give it focus.

> > Since you don't touch the models there, I'd like to set up a beta site,
> > (which would use the same database as the regular site) and use your changes
> > on that. Once we've got some feedback from core patchwork users, we can
> > merge these to the main site.
> 
> Has this been done already? Would like to test the maintainer
> functionality.

FWIW, I haven't touched anything there yet, except the navigation bar at
the top.

-- 
Damien