[Vote] Full Circle Bookshelf

[Charles Chan]

Thank you for your feedback, I will add a confirmation dialogue prior to
opening in the browser.

I have the other ideas in mind as well, but want to evolve the app
gradually.

On Thursday, July 12, 2012, Bhavani Shankar R wrote:

> On Thu, Jul 12, 2012 at 3:28 PM, Daniel Holbach
> <daniel.holbach at ubuntu.com <javascript:;>> wrote:
> > On 12.07.2012 11:45, Andrew Mitchell wrote:
> >> On Tue, Jul 10, 2012 at 03:50:49PM -0400, Michael Hall wrote:
> >>> I have reviewed this app and made a few changes to fix lintian and
> >>> arb-lint warnings.  The code can is here:
> >>>
> >>>
> https://code.launchpad.net/~mhall119/ubuntu-app-reviews/fullcircle-bookshelf/
> >>>
> >>> This app is ready for a vote.
> >>
> >> This appears to need a build-dependency on libglib2.0-bin, as it failed
> to
> >> build in a chroot without it.
> >
> > Done in lp:~dholbach/ubuntu-app-reviews/fullcircle-bookshelf and
> > uploaded to the ARB helpers PPA, so it's easier to review for the judges.
> >
>
> Packaging seems good, Built fine with libglib2.0-bin, Installs neat.
>
> Coming to the app running part:
>
> At first I thought it was a browser which enables us to browse full
> circle magazine in a window looking at the description instead a
> window opened with small cover of fcm in a window (which didnt enlarge
> while maximised) and I kept hitting left and right and tapping the
> cover thinking that nothing is happening then I opened up my browser
> to see it connected to the url of a particular issue of fcm. Would
> really like a dialogue/message saying like connecting to fcm issue xx
> through your browser when we tap on a particular issue cover to avoid
> confusion.
>
> Put it short m abstaining here. +0 for now.
>
> Regards
>
>
> --
> Bhavani Shankar
> Ubuntu Developer       |  www.ubuntu.com
> https://launchpad.net/~bhavi
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/app-review-board/attachments/20120713/3668db9a/attachment.html>