incubator-bloodhound-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Peter Koželj <pe...@digiverse.si>
Subject Re: [Apache Bloodhound] #187: Remove row count and results pagination from Dashboard
Date Wed, 19 Sep 2012 12:03:18 GMT
Hi,

attached is a patch that I am proposing. It removes the paginator but
leaves the total ticket number as is.
Any changes to the "More" link and total tickets number should probably be
addressed separately.

Peter

On Wed, Sep 19, 2012 at 1:47 PM, Joachim Dreimann <
joachim.dreimann@wandisco.com> wrote:

> I agree with displaying the total number - the More link could show "32
> More" for example to indicate how many more tickets there will be.
> Alternatively we could show something like:
>
> More  42 Total
>
> I believe we should always show the more link though, not just
> conditionally. It stands not just for 'more tickets', but also represents
> 'more query options', 'more filtering' etc - functionality that we may not
> want to duplicate on the Dashboard, and at least won't in the short term.
>
> Cheers,
> Joe
>
> On 19 Sep 2012, at 12:22, Gary Martin <gary.martin@wandisco.com> wrote:
>
> > I agree that a user might want to know the total number of tickets that
> are matching the query. I have not seen that feature in any mock-ups
> though. As it was Joe's request, I hope he might comment on this aspect.
> >
> > Meanwhile thanks for the note about the "More" link. Discussions of this
> may overlap a little with #190 which Jure is looking at. On the other hand,
> I don't think that point is covered by #190 so I would probably encourage
> you to raise a new ticket.
> >
> > As for a possible answer to it: I don't suppose it would be missed if
> the "More" link was shown conditionally. Although it could be considered
> useful as a link to the appropriate query, the alternative of changing the
> link text may just be confusing.
> >
> > Cheers,
> >    Gary
> >
> >
> > On 09/19/2012 11:26 AM, Peter Koželj wrote:
> >> I'll just remove it then.
> >>
> >> Another question: The task says we should remove total ticket number as
> >> well. Is that correct?
> >> Information about a number of all tickets seem useful.
> >>
> >> And a note: Should the "More" link really be there when there is less
> than
> >> max number of tickets?
> >>
> >> Cheers,
> >> Peter
> >>
> >> On Wed, Sep 19, 2012 at 12:03 PM, Gary Martin <gary.martin@wandisco.com
> >wrote:
> >>
> >>> If there is a way for the paginator to make sense we could consider
> >>> keeping the code. I think it would only make sense if it updated the
> >>> tickets shown on the same page but it is not clear that we want that
> at the
> >>> moment.
> >>>
> >>> I would lean towards the second option. Although it might be annoying,
> it
> >>> could always be reverted and adapted to a future behaviour.
> >>>
> >>> Cheers,
> >>> Gary
> >>>
> >>>
> >>>
> >>> On 09/19/2012 10:52 AM, Peter Koželj wrote:
> >>>
> >>>> There are two ways we can do this.
> >>>>
> >>>> 1. We can add a "show_paginator" argument to the TicketQueryWidget
> and set
> >>>> it to False for every widget instance on dashboard.
> >>>>     The plus side is that this leaves as with the option to have both
> >>>> style
> >>>> instances of the widget.
> >>>>     The negative side is that we keep some dead code if we never use
> the
> >>>> option with shown paginator.
> >>>>
> >>>> 2. The simple solution is to simply remove the pagination stuff from
> the
> >>>> widget_grid.html.
> >>>>      As far as I can tell, the widget is not used outside the
> dashboard
> >>>> anyway.
> >>>>
> >>>> Peter
> >>>>
> >>>> On Tue, Sep 18, 2012 at 1:18 PM, Apache Bloodhound <
> >>>> bloodhound-dev@incubator.**apache.org<
> bloodhound-dev@incubator.apache.org>>
> >>>> wrote:
> >>>>
> >>>>  #187: Remove row count and results pagination from Dashboard
> >>>>> --------------------------+---**-------------------
> >>>>>    Reporter:  jdreimann    |      Owner:  peter
> >>>>>        Type:  enhancement  |     Status:  assigned
> >>>>>    Priority:  minor        |  Milestone:
> >>>>>   Component:  dashboard    |    Version:
> >>>>> Resolution:               |   Keywords:
> >>>>> --------------------------+---**-------------------
> >>>>> Changes (by gjm):
> >>>>>
> >>>>>   * status:  new => assigned
> >>>>>   * owner:  nobody => peter
> >>>>>
> >>>>>
> >>>>> --
> >>>>> Ticket URL: <
> https://issues.apache.org/**bloodhound/ticket/187#comment:*
> >>>>> *3 <https://issues.apache.org/bloodhound/ticket/187#comment:3>>
> >>>>> Apache Bloodhound <https://issues.apache.org/**bloodhound/<
> https://issues.apache.org/bloodhound/>
> >>>>> The Apache Bloodhound (incubating) issue tracker
> >>>>>
> >>>>>
> >
>
>

Mime
  • Unnamed multipart/mixed (inline, None, 0 bytes)
View raw message