[aur-dev] [PATCH 1/6] Remove a boatload of inline table styles

[Dan McGee]

On Thu, Oct 20, 2011 at 10:17 AM, Lukas Fleischer
<archlinux at cryptocrack.de> wrote:
> On Thu, Oct 20, 2011 at 09:32:13AM -0500, Dan McGee wrote:
>> On Mon, Aug 22, 2011 at 7:18 PM, Dan McGee <dan at archlinux.org> wrote:
>> > Replacing with CSS styles where appropriate. A previously unused CSS
>> > style is tweaked in the stylesheet to match most of what was done via
>> > non-CSS styling.
>> >
>> > Signed-off-by: Dan McGee <dan at archlinux.org>
>> > ---
>> >
>> > Things look mostly the same on all these pages. However, there are a few
>> > exceptions- for example, the removal of the needless centering of the account
>> > search form.
>> Ping on this, and all the other patches from this day? Lukas, you had
>> them on your WIP branch, but then they all disappeared...
>
> I'm still waiting for that patch set (patch 5, actually) to be
> resubmitted. If you're too busy, just let me know and I'll look into
> that. I just thought I'd wait since you sent me a mail saying that you
> will fix and resend that patch later.

We work differently. :)

Usually if patches that don't have dependencies on each other look
good, I tend to merge them and not wait for the whole set to be
resubmitted. In this case, since it was 1/6, it shouldn't have any
dependencies. Given that I got no feedback on either the utf8 one or
this one, I either assume 1) you didn't originally see it or 2) you
forgot to merge it.

I'll be happy to fill people's inboxes with an identical patch 2
months later, but I don't want to waste time when people review it and
wonder what has changed, that's all.

-Dan