Mailing-List: contact bloodhound-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: bloodhound-dev@incubator.apache.org
Received-SPF: pass (athena.apache.org: domain of olemis@gmail.com designates
 209.85.220.175 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <5059FD38.3020105@wandisco.com>
References: <058.556d561ac8118c74aff2b0380f421667@incubator.apache.org>
	<073.4e66acd1d3eab58cf63112d9a398a95c@incubator.apache.org>
	<CAKE9Bs_=-fwppwSQD-nJgQztud+v5rqk6VOVJkbS+ZNVpZFoGg@mail.gmail.com>
	<50599857.30602@wandisco.com>
	<CAKE9Bs-G0VDhyBTBxLiWjZpxHcZwSC9gLQsthQ2kcN2DCjzCVw@mail.gmail.com>
	<5059AB01.60704@wandisco.com>
	<5423CED3-7E10-48CB-B445-937F444457D4@wandisco.com>
	<CAKE9Bs9_fZ+cGiBe7_EYFe7W6rQz3ASHzAuoyJ+bkah+9ctYDw@mail.gmail.com>
	<CAKE9Bs--=sL3nps8NQDxpqNbZj6WW=o5sGDgVts4WAhpumdjkA@mail.gmail.com>
	<5059FD38.3020105@wandisco.com>
Date: Wed, 19 Sep 2012 13:38:15 -0500
Message-ID: 
 <CAGMZAuNUuSTXrwL1bujADL2uVc=bG8zpoMyqSN+7nuB3+yRDXw@mail.gmail.com>
Subject: Re: [Apache Bloodhound] #187: Remove row count and results pagination
 from Dashboard
From: Olemis Lang <olemis@gmail.com>
To: bloodhound-dev@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1

On 9/19/12, Gary Martin <gary.martin@wandisco.com> wrote:
> Interesting. In that case, I will suggest that we always attach patches
> to tickets and refer to them as you have done here. For the smallest of
> patches, you can also choose to paste the text directly into an email -
> or indeed into a ticket comment as I did recently for #204. I will put
> this into a wiki page shortly.
>
> As for the patch, it looks like good work to me. Unless anyone else
> notices any problems I expect to commit it a bit later tonight.
>

hmmm ... I'm hoping any of my previous comments be considered . IMO we
shall not remove pagination . AFAICR there's an option for that ...

AFAICR in #80 I submitted a patch (... pending or already committed I
don't recall now ...) adding options to render Bootstrap pagination in
reports web page and built-in smaller pagination in widgets
(considering Joachim's suggestion ;) . Maybe we can follow a bit
further and parameterize page index visibility in query and report
widgets on top of the work made in there ...

About patches . I have the habit of including them in one of two forms

1.

{{{
#!diff

<patch contents>

}}}

2. Attachment having a name of the form
t<ticket_number>_r<changeset_id>_<descriptive_name>.diff

both ticket number and changeset ID are useful to know whether the
patch needs to be updated before applying it , and also to tag the
exact version modifications were tested against . This is a practice
inherited from Trac-dev itself so we might just include a link to
their patch submission guidelines wiki page or start from there and
customize it for our own usage .

-- 
Regards,

Olemis.

Blog ES: http://simelo-es.blogspot.com/
Blog EN: http://simelo-en.blogspot.com/

Featured article: