Page 7 of 14

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 13 Apr 2013, 22:52
by smeerbartje
mboinet wrote:Sorry, I was disconnected for a time, I needed to take a break for my first daughter coming to the world :-)

@WildHairZeron nice to know it's working.

@jonb it looks like the default style for a header in jQueryMobile is to let a 30% margin on both left and right. I just did a commit to fix this: https://github.com/mboinet/ttrss-mobile ... 008add2a0f

Hi mboinet, did you already consider to implement a swiping framework in your mobile webapp? Would be great! (see this post)

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 13 Apr 2013, 23:23
by mboinet
I tried to use the swipeleft & swiperight events offered by jQueryMobile but users and I weren't satisfied with this. It's still an enhancement in the pipeline: https://github.com/mboinet/ttrss-mobile/issues/24

Is the Github issue what you're implying when you're writing about a swiping framework?

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 14 Apr 2013, 07:23
by jonb
@jonb it looks like the default style for a header in jQueryMobile is to let a 30% margin on both left and right. I just did a commit to fix this: https://github.com/mboinet/ttrss-mobile ... 008add2a0f


How can I get the headers to wrap to a second line if needed? I tried adding

Code: Select all

white-space:normal;
overflow: visible; 


to that same section at the bottom of your revised main.css but I'm not seeing any difference.

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 14 Apr 2013, 19:27
by mboinet
The container doesn't have enough height. You should use a smaller font size en reduce top & bottom margins. But with a little test, all the header collapse if content is on one line. So you also have to put a fixed height to the header I believe.

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 14 Apr 2013, 23:44
by jonb
The container doesn't have enough height. You should use a smaller font size en reduce top & bottom margins. But with a little test, all the header collapse if content is on one line. So you also have to put a fixed height to the header I believe.

Shouldn't it be easier to increase the height of the container? How would I do that? Where is the height set?

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 16 Apr 2013, 09:38
by jdelamater99
mboinet wrote:Sorry, I was disconnected for a time, I needed to take a break for my first daughter coming to the world :-)


BTW, congrats!

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 16 Apr 2013, 17:37
by mboinet
jonb wrote:Shouldn't it be easier to increase the height of the container? How would I do that? Where is the height set?

This is set nowhere yet. You have to find the place to do that. I'll let other people help you on this.

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 17 Apr 2013, 21:38
by jdelamater99
I had issues getting this to work on my server, and I thought it was something I was doing wrong in the config file. I kept bashing my head against the desk trying to figure out how it needed to be setup to work. Then I dug into the index.html file, and noticed a few lines were missing part of the url, the "http:" part to be exact. Made those changes and it now works for me. thanks!

update

Code: Select all

<link rel="stylesheet" href="//cdnjs.cloudflare.com/ajax/libs/jquery-mobile/1.3.0/jquery.mobile.min.css" />
  <link rel="stylesheet" href="main.css">
  <script src="//cdnjs.cloudflare.com/ajax/libs/jquery/1.9.1/jquery.min.js"></script>
  <script src="//cdnjs.cloudflare.com/ajax/libs/underscore.js/1.4.4/underscore-min.js"></script>
  <script src="//cdnjs.cloudflare.com/ajax/libs/backbone.js/1.0.0/backbone-min.js"></script>
  <script src="conf.js"></script>
  <script src="main.js"></script>
  <script src="//cdnjs.cloudflare.com/ajax/libs/jquery-mobile/1.3.0/jquery.mobile.min.js"></script>


with

Code: Select all

<link rel="stylesheet" href="http://cdnjs.cloudflare.com/ajax/libs/jquery-mobile/1.3.0/jquery.mobile.min.css" />
  <link rel="stylesheet" href="main.css">
  <script src="http://cdnjs.cloudflare.com/ajax/libs/jquery/1.9.1/jquery.min.js"></script>
  <script src="http://cdnjs.cloudflare.com/ajax/libs/underscore.js/1.4.4/underscore-min.js"></script>
  <script src="http://cdnjs.cloudflare.com/ajax/libs/backbone.js/1.0.0/backbone-min.js"></script>
  <script src="conf.js"></script>
  <script src="main.js"></script>
  <script src="http://cdnjs.cloudflare.com/ajax/libs/jquery-mobile/1.3.0/jquery.mobile.min.js"></script>

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 18 Apr 2013, 11:53
by mboinet
It's on purpose: http://paulirish.com/2010/the-protocol-relative-url/

To quote Cloudfare :
Supports http/https/spdy, we encourage leaving the protocol off!


This soon will be irrelevant, I'm working on optimization with require.js (require branch on Github) and all scripts (the goal is only one) will be stored on the same server.

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 18 Apr 2013, 11:57
by smeerbartje
Mboinet... not really somehting to say, except: "GOOD JOB"... I really love your mobile app on my iOS devices.

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 18 Apr 2013, 15:57
by xtaz
Looks like the API limit has been increased to 200 instead of 60. Can the max setting also be changed to reflect this?

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 18 Apr 2013, 16:59
by jdelamater99
mboinet wrote:It's on purpose: http://paulirish.com/2010/the-protocol-relative-url/

To quote Cloudfare :
Supports http/https/spdy, we encourage leaving the protocol off!


This soon will be irrelevant, I'm working on optimization with require.js (require branch on Github) and all scripts (the goal is only one) will be stored on the same server.


Good to know. And I swear it wasn't working on my site, but when I switched it back to get a screenshot of what it was doing, it worked. So, maybe it was a caching issue, and the browser was hanging on to an incorrect stored version of conf.js. Oh well, working now. Thanks.

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 18 Apr 2013, 18:05
by mboinet
xtaz wrote:Looks like the API limit has been increased to 200 instead of 60. Can the max setting also be changed to reflect this?


Done with this commit: https://github.com/mboinet/ttrss-mobile ... c6eb463667

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 22 Apr 2013, 09:34
by smeerbartje
Hi, I found a small bug: the source feed name doesn't load... at least not on my server. This happens to all the feeds.

Image

Re: Mobile webapp for TT RSS: ttrss-mobile

Posted: 22 Apr 2013, 11:50
by mboinet
It's a known bug: https://github.com/mboinet/ttrss-mobile/issues/37

I'll try to fix this soon.