cragjagged 44 Posted January 27, 2015 Posted January 27, 2015 Hi everybody! I've recently been having a play around with the source for the app and I've made a couple of alterations to some of the views that I was interested in getting peoples thoughts on. Series overview now shows date range of the series in the same fashion as the web client (so will display 2001-present for current series), and replaces the number of episodes with the number of seasons. Season overview, the first line shows series level info (Name, rating, episode length), the second line shows the season level info (name, season year, total episodes), the overview text shows the season overview instead of the series overview too. I'm not too sure on this one as it's a bit cluttered (even if the grouping made sense to me at the time), also I've found the metadata for season overview and year to be a little inconsistent 9
Luke 42077 Posted January 27, 2015 Posted January 27, 2015 Well done! Great to see the participation. 1
chessdragon136 677 Posted January 27, 2015 Posted January 27, 2015 First of all, it's great to have another contributor The first screen I really like, it just seems right (did you fix the star positioning??) - The only thing to add is to make the text blue to be consistent with the rest of the pages. The second screen, perhaps make the metadata on the first line in a smaller font? So it's not as detracting from the title.
cragjagged 44 Posted January 29, 2015 Author Posted January 29, 2015 (edited) Thanks for the encouragement everyone! I'm still working on a few of the views to try and get something that's consistent and doesn't feel too cluttered. But I thought i'd share this little sneak peak i'm working on Unwatched item counters Edited January 29, 2015 by cragjagged 5
CBers 7450 Posted January 29, 2015 Posted January 29, 2015 Is that Unwatched across the show? Can it be per season?
chessdragon136 677 Posted January 29, 2015 Posted January 29, 2015 Looks great - Just a though as I know its been requested and you're looking into this area - could you add metadata if a film/show is in 3D, just text will do. Do you want to work on your own fork and put pull requests when ready or would you like access to the master branch to work off directly?
cragjagged 44 Posted January 29, 2015 Author Posted January 29, 2015 @@CBers that's unwatched per season overlaying the season image. The text in blue is unwatched for whole series (my local development version is a bit of a mess at the moment hence the disconnect). I'm trying to get the views displaying info in as similar a way as possible as the other clients, I'm using the web client and MBT as a reference to try and get a consistent look and feel. For the moment I'm working with the views that are already in the app and just making minor changes to the meta data that's been displayed. @@chessdragon136 I'll take a look at 3D this evening assuming I don't get snowed in at work! I have a fair bit of 3D content but oddly I've never played any through the app so not clear on what needs to be done yet, but I'm sure it'll be obvious. For the time being I'd prefer to work off my own fork as I'm a bit new to this and whilst I appreciate merging in my pulls further down the line might be more work, it's probably going to be less work than fixing anything I break in the master repository!
chessdragon136 677 Posted January 29, 2015 Posted January 29, 2015 Playback through the app is hit and miss - still a major area to investigate, but at the moment I don't think there is any visual markers that show if content is 3D which there needs to be As for merging, take your time. I like what you're doing so just let me know and if you have queries on any code please ask. I know its not the best commenting in the world but most pages follow a rather similar pattern.
FrostByte 5392 Posted January 29, 2015 Posted January 29, 2015 I ripped Sin City a Dame to Kill For and there is a blue 3D symbol in the bottom left corner where the 5.1, x264, etc symbols are
chessdragon136 677 Posted January 29, 2015 Posted January 29, 2015 I ripped Sin City a Dame to Kill For and there is a blue 3D symbol in the bottom left corner where the 5.1, x264, etc symbols are Forgot about that!! - Still think there needs to be a 3D text marker on the all films views 2
cragjagged 44 Posted January 30, 2015 Author Posted January 30, 2015 Not sure on the 3d one to be honest, I use coverart to overlay a 3d logo already. Which shows in the all movies view for definite, and possibly in the details view too. Then there's the media info style icon on the detail page too. Just need to be careful about showing the same information over and over again and cluttering the interface. I'll have another look at it tomorrow and maybe mock a couple of ideas up. I've nearly finished the rest of my changes too, hopefully I'll be able to put some comparison screenshots up tomorrow 2
cragjagged 44 Posted January 31, 2015 Author Posted January 31, 2015 OK new view layouts are below for feedback. The smaller screengrabs are as the relevant sections are in the current version, the larger screengrabs are my suggestions. There's nothing too radical to be honest just small adjustments here and there. Missing episodes are now being shown too, and i've also got a watched/unplayed count overlay in the works as shown in a previous screenshot but it currently needs a bit more work to get the alignment working in all the views. 2
chessdragon136 677 Posted January 31, 2015 Posted January 31, 2015 (edited) Yay I like - It does definitely work and ties in with the other clients nicely too. I really can't say how happy i am to have someone else looking at this. Especially as I'm rubbish with layouts, design in general! Thanks Only thing, the red missing icon - whats that indicating? I should learn to read. Maybe its me but I think its a bit too distracting from the other content, eyes are drawn to that instead of the images / info. More worryingly, your screenshots show the missing episodes as playable. I think its because i've never actually coded it in to display missing / upcoming episodes. But looks like you've got it in the works but it change the item page somewhat. Edited January 31, 2015 by chessdragon136
cragjagged 44 Posted February 1, 2015 Author Posted February 1, 2015 Yeah missing and upcoming episodes do currently come up as playable, I was going to take a shot at that next to tie in with flagging the missing episodes. It's a quick fix to disable the play button on them but then it breaks the left/right key functionality to move between episodes so will need a bit more tweaking to keep the experience consistent. I can understand that you can see the missing icon being a bit distracting though, I just wanted an indicator that the episode was missing so just dropped it in where the watched indicator was showing. If you look at the web client and MBT it's handled slightly differently due to the different view they have. Web client overlays the info onto the images, but doesn't have a similar list style view. MBT which does have a similar list style view handles it in the same way as I've got it here albeit with green backgrounds rather than red. Personally I prefer how it's handled in MBT, but that might just be my personal preference. Also if you have your profile configured to not show missing and unaired episodes it removes them all from view anyway, the only drawback is the setting isn't exposed in this app so you have to use the webclient to change it. 1
CBers 7450 Posted February 1, 2015 Posted February 1, 2015 LooksLooks good on my phone, but will check it out better on my PC later. I wouldn't want to see missing/upcoming episodes, so if they can be hidden....
chessdragon136 677 Posted February 1, 2015 Posted February 1, 2015 As part of the rework I'll make some of the server settings alterable from the app (if i can, pretty sure i can) As for the missing icon, i was only thinking of adjusting the colour to be a bit darker so its not so distracting from the other content, apart from that i like the changes. Is upcoming marked the same way? Thanks
im85288 1493 Posted February 1, 2015 Posted February 1, 2015 @@cragjagged - great work there, well done
chessdragon136 677 Posted February 1, 2015 Posted February 1, 2015 Ok I've finally got around to altering the Settings page to be a lot more dynamic and I have now included a Server settings section, which will expose the unaired / missing settings from within the app. 3
cragjagged 44 Posted February 2, 2015 Author Posted February 2, 2015 OK so I've taken another shot at the below page and reworked the layout a bit more. This is my personal favourite so far (and is also more or less the same as MBT) 4
chessdragon136 677 Posted February 2, 2015 Posted February 2, 2015 You're churning out some really nice themes and I really want to get them into the app! My critique on this one - Make the unaired / missing boxes curved and not rectangular -think it may take the edge off it and probably looks better with the watched circle. 1
MrWebsmith 598 Posted February 5, 2015 Posted February 5, 2015 these new views are making me want to jump back into this app and see if i can leave mbc behind yet.. nice work! 2
chessdragon136 677 Posted February 5, 2015 Posted February 5, 2015 these new views are making me want to jump back into this app and see if i can leave mbc behind yet.. nice work! The next build will be impressive 5
cragjagged 44 Posted February 5, 2015 Author Posted February 5, 2015 The next build will be impressive No pressure then 2
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now