Bug#708838: developers.php: Hide or sort differently co-maintained packages

[Joachim Breitner]

Package: qa.debian.org
Severity: wishlist

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi,

most of the packages listed on my page at
http://qa.debian.org/developer.php?login=nom...@debian.org are
team-maintained; when I want to get information about them I check the
team’s developer page. But I have trouble finding my own packages in the
list.

Please consider sorting co-maintained packages in a category of their
own (comparable to “Sponsored/other uploads”), or add an option to hide
them.

As always: Please indicate if you agree with the bug even if you don’t
implement it right away, as it would encourage patches.

Greetings,
Joachim


- -- System Information:
Debian Release: jessie/sid
APT prefers unstable
APT policy: (500, 'unstable'), (101, 'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.8-1-amd64 (SMP w/4 CPU cores)
Locale: LANG=de_DE.UTF-8, LC_CTYPE=de_DE.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)

iEYEARECAAYFAlGX+k8ACgkQ9ijrk0dDIGxmDgCdEbu6Al3FyKUIIVXlIjaikns4
0VUAoJ9jaIGypw5L/XGALjm6HUyPVK0t
=mOQH
-----END PGP SIGNATURE-----


--
To UNSUBSCRIBE, email to debian-bugs-...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listm...@lists.debian.org

[Paul Wise]

Control: user qa.deb...@packages.debian.org
Control: usertags -1 + ddpo

On Sun, May 19, 2013 at 6:01 AM, Joachim Breitner wrote:

> As always: Please indicate if you agree with the bug even if you don’t
> implement it right away, as it would encourage patches.

I expect this would be a popular feature. I would suggest making it an
option rather than the default though.

--
bye,
pabs

http://wiki.debian.org/PaulWise

[Joachim Breitner]

Hi,

Am Sonntag, den 19.05.2013, 11:22 +0800 schrieb Paul Wise:
> I expect this would be a popular feature. I would suggest making it an
> option rather than the default though.

I just found out that the option already exists:
http://qa.debian.org/developer.php?login=nomeata&comaint=no
but it is only accessible from http://qa.debian.org/developer.php (where
I almost never go) and not from the „Display configuration“ (which is
also hard to find, as it is below a long table).

So my wishlist request is to add the comaint option to the table at the
bottom, maybe with a forth option to show the comaintained packages
separately.

I would have worked on this myself, but it seems that
http://quantz.debian.org/~nomeata/developer.php or
http://qa.debian.org/~nomeata/developer.php does not work. If it would
it would be easier for occasional contributors to send in (tested)
patches.

Greetings,
Joachim

--
Joachim "nomeata" Breitner
Debian Developer
nom...@debian.org | ICQ# 74513189 | GPG-Keyid: 4743206C
JID: nom...@joachim-breitner.de | http://people.debian.org/~nomeata

[Paul Wise]

On Sun, May 19, 2013 at 5:31 PM, Joachim Breitner wrote:

> I would have worked on this myself, but it seems that
> http://quantz.debian.org/~nomeata/developer.php or
> http://qa.debian.org/~nomeata/developer.php does not work. If it would
> it would be easier for occasional contributors to send in (tested)
> patches.

Looks like Bart Martens added a symlink for you so the second URL does work now.

[Joachim Breitner]

Hi,

Am Sonntag, den 19.05.2013, 11:22 +0800 schrieb Paul Wise:

> Control: user qa.deb...@packages.debian.org
> Control: usertags -1 + ddpo
>
> On Sun, May 19, 2013 at 6:01 AM, Joachim Breitner wrote:
>
> > As always: Please indicate if you agree with the bug even if you don’t
> > implement it right away, as it would encourage patches.
>
> I expect this would be a popular feature. I would suggest making it an
> option rather than the default though.

I worked a bit on the page; see the attached patches (which make minor
other adjustments as well).

I did not yet add an option to show comaintained packages in a separate
table. Would that be desirable?

[Joachim Breitner]

Hi,

I did some more cosmetic polishing (I hope I managed to stop before it
became bikeshedding).

[Paul Wise]

On Mon, May 20, 2013 at 2:58 AM, Joachim Breitner wrote:

> I did not yet add an option to show comaintained packages in a separate
> table. Would that be desirable?

I expect that would be useful to some folks.

[Joachim Breitner]

Hi,

Am Montag, den 20.05.2013, 09:20 +0800 schrieb Paul Wise:
> On Mon, May 20, 2013 at 2:58 AM, Joachim Breitner wrote:
>
> > I did not yet add an option to show comaintained packages in a separate
> > table. Would that be desirable?
>
> I expect that would be useful to some folks.

ok, I’ll work on it as soon as the other patches have been reviewed and
merged (to avoid having to re-do the separate table work afterwards).

[Joachim Breitner]

Hi,

Am Montag, den 20.05.2013, 21:11 +0200 schrieb Jakub Wilk:
> * Joachim Breitner <nom...@debian.org>, 2013-05-19, 20:58:
> >+ if (!strcmp($comaint, "no")) {
> >+ $maintainer_data = "Maintainer:";
> >+ } else if (!strcmp($comaint, "only")) {
> >+ $maintainer_data = "Co-Maintainer:";
> >+ } else {
> >+ $maintainer_data = "Maintainer and Co-Maintainer:";
> >+ }
>
> Any particular reason to use capital letters in "Co-Maintainer"?

not really; feel free to fix it while applying.
(seems to be too minor to rebase and resend the patches)

[Jakub Wilk]

* Joachim Breitner <nom...@debian.org>, 2013-05-19, 20:58:
>+ if (!strcmp($comaint, "no")) {
>+ $maintainer_data = "Maintainer:";
>+ } else if (!strcmp($comaint, "only")) {
>+ $maintainer_data = "Co-Maintainer:";
>+ } else {
>+ $maintainer_data = "Maintainer and Co-Maintainer:";
>+ }

Any particular reason to use capital letters in "Co-Maintainer"?

--
Jakub Wilk

[Paul Wise]

On Mon, May 20, 2013 at 5:31 PM, Joachim Breitner wrote:

> ok, I’ll work on it as soon as the other patches have been reviewed and
> merged (to avoid having to re-do the separate table work afterwards).

Patches reviewed, merged and deployed.

Also thanks for motivating me to get a git-svn checkout.

[Joachim Breitner]

Hi,

Am Donnerstag, den 23.05.2013, 22:06 +0800 schrieb Paul Wise:
> On Mon, May 20, 2013 at 5:31 PM, Joachim Breitner wrote:
> > ok, I’ll work on it as soon as the other patches have been reviewed and
> > merged (to avoid having to re-do the separate table work afterwards).
>
> Patches reviewed, merged and deployed.

thanks! Let me know if my code causes any trouble.

[Paul Wise]

On Thu, 2013-05-23 at 23:27 +0200, Joachim Breitner wrote:

> thanks! Let me know if my code causes any trouble.

From IRC:

<raphael> pabs: always displaying the config div of the ddpo is quite
annoying on "small" screens

I'm inclined to agree, could you make a patch that hides it by default
but shows it if the screen is large enough to display say 80% of the
settings?

Even better would be to implement a fluid column layout for the settings
section of the page, based on something like this:

http://twitter.github.io/bootstrap/scaffolding.html#gridSystem

[Paul Wise]

On Fri, 2013-05-24 at 09:08 +0200, Joachim Breitner wrote:

> I assume he was talking about the width of the screen (as the height is
> scrolled for almost every multi-package maintainer anyways), where the
> settings have caused the scrollbar to appear?

I assumed likewise.

> That’s a valid point, and the linked grid system would be an option,
> or more simply a different layout that has one setting per row instead
> of per column.

One setting per row sounds like it would waste vertical space so if you
can figure out the fluid column system I linked to, that would be good.

[Joachim Breitner]

Hi,

I assume he was talking about the width of the screen (as the height is
scrolled for almost every multi-package maintainer anyways), where the

settings have caused the scrollbar to appear? That’s a valid point, and

the linked grid system would be an option, or more simply a different
layout that has one setting per row instead of per column.

[Joachim Breitner]

Hi,

Am Freitag, den 24.05.2013, 15:11 +0800 schrieb Paul Wise:
> One setting per row sounds like it would waste vertical space so if you
> can figure out the fluid column system I linked to, that would be good.

fluid layout implemented, see the attached patch (and
http://qa.debian.org/~nomeata/developer.php?packages=ghc for a preview).

[Debian Bug Tracking System]

Your message dated Sun, 26 May 2013 15:30:08 +0800
with message-id <CAKTje6FJFZiu47pH_8hpWE4t...@mail.gmail.com>
and subject line Re: Bug#708838: developers.php: Hide or sort differently co-maintained packages
has caused the Debian Bug report #708838,
regarding developers.php: Hide or sort differently co-maintained packages
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact ow...@bugs.debian.org
immediately.)


--
708838: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=708838
Debian Bug Tracking System
Contact ow...@bugs.debian.org with problems