john-tornblom / TVHGuide

An Android client for TVHeadend using the HTSP protocol
GNU General Public License v3.0
44 stars 24 forks source link

UI Update #3

Closed etmtvdp closed 12 years ago

etmtvdp commented 12 years ago

Hey John,

Here are the files for the UI update...

Had to spend some time in the java-classes for the UI to get it all to work....

Added a generic "utility" class (to help support re-formatting the dates, so when it's "today", the date shows "today" or "tomorrow", which (imho) is a lot nicer and easier to read for end-users...

I have previously built a XBMC control-GUI for GBPVR (a Windows based PVR solution), and I had that in there as well, makes things just a little easier...

I didn't spend much time on the "standard" listactivities, but more on the details-screens and the top-header for the programme-section. The "eye" icon (I included the svg-file I used to generate it), is actually a button that starts the player when clicked....

Have a look 'round... I pulled the "external player" support and will start playing with the fragmenst and actionbar, drop me a mail if you have questions / issues; I'll drop you a mail when I've played 'round with fragments etc. some more, and hopefully give you some screenshots for tablet UI...

john-tornblom commented 12 years ago

I seem to be getting java.lang.NullPointerException in the programme list. Have you tested so that everything works?

2011/10/10 etmtvdp < reply@reply.github.com>

Hey John,

Here are the files for the UI update...

Had to spend some time in the java-classes for the UI to get it all to work....

Added a generic "utility" class (to help support re-formatting the dates, so when it's "today", the date shows "today" or "tomorrow", which (imho) is a lot nicer and easier to read for end-users...

I have previously built a XBMC control-GUI for GBPVR (a Windows based PVR solution), and I had that in there as well, makes things just a little easier...

I didn't spend much time on the "standard" listactivities, but more on the details-screens and the top-header for the programme-section. The "eye" icon (I included the svg-file I used to generate it), is actually a button that starts the player when clicked....

Have a look 'round... I pulled the "external player" support and will start playing with the fragmenst and actionbar, drop me a mail if you have questions / issues; I'll drop you a mail when I've played 'round with fragments etc. some more, and hopefully give you some screenshots for tablet UI...

You can merge this Pull Request by running:

git pull https://github.com/etmtvdp/TVHGuide master

Or you can view, comment on it, or merge it online at:

https://github.com/john-tornblom/TVHGuide/pull/3

-- Commit Summary --

  • External Player Support
  • Merge remote-tracking branch 'upstream/master'
  • Intermediate version;
  • Mostly finished new layout;
  • Updated Layout
  • Get "today" and "tomorrow" values from strings.xml
  • Clean update

-- File Changes --

M AndroidManifest.xml (2) A assets/eye.svg (173) A res/drawable/button.svg (410) A res/drawable/eye.png (0) A res/drawable/timebar_background.xml (13) A res/drawable/timebar_foreground.xml (13) M res/layout/ch_widget.xml (48) M res/layout/pr_layout.xml (168) M res/layout/pr_widget.xml (118) A res/layout/pr_widget_listitem.xml (77) A res/layout/pr_widget_topitem.xml (87) M res/layout/rec_layout.xml (141) M res/layout/rec_widget.xml (1) M res/values/strings.xml (17) A res/values/styles.xml (9) M res/xml/preferences.xml (1) M src/org/me/tvhguide/ChannelListActivity.java (63) M src/org/me/tvhguide/ProgrammeActivity.java (25) M src/org/me/tvhguide/ProgrammeListActivity.java (241) M src/org/me/tvhguide/RecordingActivity.java (36) M src/org/me/tvhguide/RecordingListActivity.java (8) M src/org/me/tvhguide/htsp/HTSConnection.java (1) M src/org/me/tvhguide/htsp/HTSMessage.java (17) M src/org/me/tvhguide/htsp/HTSService.java (8) A src/org/me/tvhguide/util/Util.java (61)

-- Patch Links --

https://github.com/john-tornblom/TVHGuide/pull/3.patch https://github.com/john-tornblom/TVHGuide/pull/3.diff

Reply to this email directly or view it on GitHub: https://github.com/john-tornblom/TVHGuide/pull/3

etmtvdp commented 12 years ago

Hey John,

Haven't seen that myself. Any more info?

I'm running that version on my tablet... Oh, wait, maybe I left a reference to the "external player"...

I'll look tomorrow.

//Ton Op 16 okt. 2011 17:49 schreef "john-tornblom" < reply@reply.github.com> het volgende:

etmtvdp commented 12 years ago

Hey John,

Just checked, but I don't get that (and I see that I removed the external-player stuff)....

Do you get this on the programme-list or on programme-details?

//Ton

On Mon, Oct 17, 2011 at 9:26 PM, Ton van der Poel < ton.van.der.poel@gmail.com> wrote:

Hey John,

Haven't seen that myself. Any more info?

I'm running that version on my tablet... Oh, wait, maybe I left a reference to the "external player"...

I'll look tomorrow.

//Ton Op 16 okt. 2011 17:49 schreef "john-tornblom" < reply@reply.github.com> het volgende:

john-tornblom commented 12 years ago

I got it in the programme-list, the channelName TextView was not found ;( . I will try to merge the UI again, perhaps I missed something.