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 gary.martin@wandisco.com
 designates 74.125.83.47 as permitted sender)
Message-ID: <505C96CE.2070207@wandisco.com>
Date: Fri, 21 Sep 2012 17:33:18 +0100
From: Gary Martin <gary.martin@wandisco.com>
User-Agent: Mozilla/5.0 (X11; Linux x86_64;
 rv:15.0) Gecko/20120827 Thunderbird/15.0
MIME-Version: 1.0
To: bloodhound-dev@incubator.apache.org
Subject: Re: [Apache Bloodhound] #187: Remove row count and results pagination
 from Dashboard
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>
 <CAGMZAuNUuSTXrwL1bujADL2uVc=bG8zpoMyqSN+7nuB3+yRDXw@mail.gmail.com>
 <505A5347.80503@wandisco.com>
 <CAGMZAuPwzaRZfuoXe1=iVUUcjPF+m+=9XWZMZq+2kdb17csgiA@mail.gmail.com>
 <CAGMZAuPQUmM1WgWgymCAogUX_aTZvtdgVRYh4YY-GghpZrTsVA@mail.gmail.com>
In-Reply-To: 
 <CAGMZAuPQUmM1WgWgymCAogUX_aTZvtdgVRYh4YY-GghpZrTsVA@mail.gmail.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

On 09/20/2012 02:57 PM, Olemis Lang wrote:
> On 9/20/12, Olemis Lang <olemis@gmail.com> wrote:
>> On 9/19/12, Gary Martin <gary.martin@wandisco.com> wrote:
>>> On 19/09/12 19:38, Olemis Lang wrote:
>>>> 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 ...
>>> Well, it is entirely possible that I have missed the scope over which
>>> these changes apply. Do we have widgets where a change of page updates
>>> the content of the widget itself?
>> so far we don't . But I was considering this to be the case in the
>> future .
> ... considering Joachim's suggestions . I mean Edit and List buttons
> included in widget header in mockups e.g. [1]_
>
> .. [1] MultiSite - Apache Bloodhound
>          (http://svn.apache.org/repos/asf/incubator/bloodhound/trunk/doc/html-templates/product.html)
>

It looks to me like #80 does not provide all that much other than a 
change from trac defaults in pagination to bootstrap. It doesn't look 
like this change has been agreed to because there was a feeling that the 
pagination was a bit too intrusive too. That means that all I can do 
with that ticket is move the template to the new suggested location.

Meanwhile, I am not sure how the edit and list buttons are relevant to 
this part of the discussion although I suppose could be better ways of 
suggesting what the "more" link is for.. maybe.

Cheers,
     Gary