← Back to team overview

ubuntu-phone team mailing list archive

Re: [Ubuntu-touch-coreapps] [RSS Reader] Index0 and 1?

 

Hi everybody!

Svenn, thanks for your comments! All of them are reasonable.

1. Experiment only :) I like CheckBox too, it's looks awesome!

2. Let's all think over! I really want to hear comments from Lisette.

3. Forgot one little thing! Yes, it works well, but only for feeds with
status '1' (append new feed via my pages and try to edit it. Joey will fix
it, he don't uses field "status" in DB on which I rely)

And look - one page contains two methods to add feeds (toolbar and button).
I left both of them. We can decide their future in discussion. And a lot of
other things, like must feed can be binded to many topics or not. And so
on. I've a lot of questions, which I can't solve only myself :)

Have a good day,
Roman!


2013/7/17 Svenn-Arne Dragly <s@xxxxxxxxxx>

> Great work, Roman!
>
> It looks really nice and felt very intuitive to use.
>
> A few comments, though:
>
> 1. You are using CheckBox in the Add Topic view to select feeds, while
> we use Switch in the Add Feed view to select topics. Perhaps both
> should use the same component? Personally I prefer the CheckBox.
>
> 2. Also, I think long press + Remove on a feed should remove it from
> the topic only. If it exists in only one topic, it could be removed
> completely.
>
> 3. By the way, editing an existing feed seems not to work. Is it
> implemented yet?
>
> I have split the new pages into separate files and changed from Switch
> to CheckBox in the following branch. Feel free to merge it into your
> branch or to the alpha-branch:
>
> lp:~dragly/ubuntu-rssreader-app/topics-management-branch
>
>
>
> Best regards,
> Svenn-Arne
>
> On Tue 16 Jul 2013 10:59:37 PM CEST, Roman Shchekin wrote:
> > And here you are, folks, my topics and feeds management for alpha:
> > lp:~mrqtros/ubuntu-rssreader-app/topics-management-branch
> >
> > I hope you like it! (Keep in mind that some solutions are results of
> > sdk issues).
> >
> > P.S. Joey, especially to you - look, I've added 5 pages (4 complete
> > and 1 just sketch), separate them to files please as you want and
> > merge with you alpha version. Am I so tired to do this today myself
> > and I have a lot of businesses tomorrow :(
> > As you can see - I used deep navigation - as result our app become
> > extremely clear and simple, but in the same time very useful!
> >
> >
> > 2013/7/17 Roman Shchekin <mrqtros@xxxxxxxxx <mailto:mrqtros@xxxxxxxxx>>
> >
> >     Hi all!
> >
> >     Lisette, where we can find icons? :)
> >     We are still using the same "avatar.png" for all toolbar items, I
> >     already hate it :D
> >     Please, make me happy :))
> >
> >     BR,
> >     Roman!
> >
> >
> >     2013/7/16 Joey Chan <qqworini@xxxxxxxxx <mailto:qqworini@xxxxxxxxx>>
> >
> >         Opsss...  just two button for testing, forgot to remove them   :P
> >
> >
> >         2013/7/16 Lisette Slegers <lisette.slegers@xxxxxxxxxxxxx
> >         <mailto:lisette.slegers@xxxxxxxxxxxxx>>
> >
> >             Hi Joey!
> >
> >             I was just looking at the latest RSS reader branch (which
> >             is awesome!), and I was wondering about 2 toolbar actions
> >             in the article view: Index 0 and Index 1. See
> >             screenshot:
> https://docs.google.com/file/d/0B5ZePTRp2C_JTm9UaFJua05QXzQ/edit?usp=sharing
> >
> >             What are they?
> >
> >             Thanks,
> >             Lisette
> >
> >
> >
> >         --
> >         Mailing list: https://launchpad.net/~ubuntu-touch-coreapps
> >         Post to     : ubuntu-touch-coreapps@xxxxxxxxxxxxxxxxxxx
> >         <mailto:ubuntu-touch-coreapps@xxxxxxxxxxxxxxxxxxx>
> >         Unsubscribe : https://launchpad.net/~ubuntu-touch-coreapps
> >         More help   : https://help.launchpad.net/ListHelp
> >
> >
> >
> >
> >
>

Follow ups

References