meteor-famous-views icon indicating copy to clipboard operation
meteor-famous-views copied to clipboard

WIP, fview-demo

Open gadicc opened this issue 10 years ago • 17 comments

Work in progress. Simple app using famous-views. Mobile-first, end goal is fully responsive for all sizes.

Currently best viewed on mobile or in mobile emulation: http://fview-demo.meteor.com/ https://github.com/gadicc/fview-demo

@trusktr and @PEM--, thought you might be interested to see this, especially the total 60 lines of templating all in Jade. The JS weighs in at 70 lines, but would be a lot shorter in CS, I admit :>

Demos off two cool new things in the new version released today, reactive properties on scrollview (unfortunately the pageChange even is a bit borked, but I worked around it a little), and spacebar text inside an attribute value. Neither is documented yet but they're mentioned in the History and seen in use here.

gadicc avatar Nov 08 '14 14:11 gadicc

Nice. Love the idea of a real world demo with RWD in mind.

Could I add the following minor adjustments to it or do you prefer experimenting stuff on your own?

  • Remove the CSS to for a drop in smaller replacement of styles in CS: This speeds up the initial screen loading on mobile and allow sharing value between the logic and the styles.
  • Turn your JS in CS.

PEM-- avatar Nov 08 '14 14:11 PEM--

For the hamburger button, there is this really cool animation that @IjzerenHein did: https://github.com/IjzerenHein/famous-animatedIcon

PEM-- avatar Nov 08 '14 14:11 PEM--

Happy for you to contribute :) Our minor collaboration has been pretty fun so far. Added you as a collaborator. I guess by the end of it you'll have me coding natively in CS haha.

Ah, that's cool... yeah I rate we can squeeze that in. I'm out of time for today but will try integrate it tomorrow/monday.

gadicc avatar Nov 08 '14 15:11 gadicc

Looks sweet! Just needs to fill my whole screen on Nexus 5 Chrome. xD

Do email attachments work? Do you see the screenshot?

/#!/JoePea On Nov 8, 2014 7:09 AM, "Gadi Cohen" [email protected] wrote:

Happy for you to contribute :) Our minor collaboration has been pretty fun so far. Added you as a collaborator. I guess by the end of it you'll have me coding natively in CS haha.

Ah, that's cool... yeah I rate we can squeeze that in. I'm out of time for today but will try integrate it tomorrow/monday.

— Reply to this email directly or view it on GitHub https://github.com/gadicc/meteor-famous-views/issues/132#issuecomment-62260951 .

trusktr avatar Nov 08 '14 18:11 trusktr

Nope. No screenshot.

PEM-- avatar Nov 08 '14 18:11 PEM--

@PEM-- ah, here it is: shot

trusktr avatar Nov 08 '14 18:11 trusktr

Ho ho, this one is funny :smile:

PEM-- avatar Nov 08 '14 19:11 PEM--

Deadly on Nexus 5. I've just checked on my midp (an old HTC Desire C) and it works. Pretty weird. I use this device as I know it is normally the worst case scenario (you should see my guys when I test their apps on it, it is always a funny moment).

PEM-- avatar Nov 08 '14 19:11 PEM--

Haha. Got a screenshot?

trusktr avatar Nov 08 '14 19:11 trusktr

screenshot_2014-11-08-20-53-10

It works on the stock browser as well as Chrome. I'm charging my Android tablet to try reproducing your issue.

PEM-- avatar Nov 08 '14 19:11 PEM--

@gadicc I've updated the sources. Now everything is in CS.

There is still some stuff that we should take care of: CSSRules are still contained in famous-views as CSS files. Thus, I can't remove the last bits of calls.

PEM-- avatar Nov 08 '14 20:11 PEM--

I've removed the call to famous.polyfills so that users can call it themselves as stated in the documentation (Quickstart / General help).

PEM-- avatar Nov 08 '14 20:11 PEM--

In the view-demo, when the animation of the icons takes place, there is a little glitch: the main window is paused for at least one frame. It breaks its fluidity. I think that the reactive event on Template.homePageIndicator.helpers is not synchronized with Famo.us's Engine. It should be called with an Engine.nextTick.

PEM-- avatar Nov 08 '14 20:11 PEM--

:-1: Dawn, I can't reproduce @trusktr issue with his Nexus 5. I've tried an Android tablet (a Samsung Galaxy Tab) and it flies like native. @trusktr could you help us on this?

PEM-- avatar Nov 08 '14 20:11 PEM--

Haven't had time this weekend for fun stuff. Just work. :'( I'll be back to try and see why this happens. Of thing is I tried this again today and it randomly changes size, so it's sometimes a little bigger, but still not full window.

/#!/JoePea On Nov 8, 2014 12:47 PM, "Pierre-Eric Marchandet" [email protected] wrote:

[image: :grin:] Dawn, I can't reproduce @trusktr https://github.com/trusktr issue with its Nexus 5. I've tried an Android tablet and it flies like native. @trusktr https://github.com/trusktr could you help us on this?

— Reply to this email directly or view it on GitHub https://github.com/gadicc/meteor-famous-views/issues/132#issuecomment-62278157 .

trusktr avatar Nov 10 '14 08:11 trusktr

No worries, also only have very limited time to work on this, but hopefully bit by bit it will amount to something.

@PEM-- , thanks for all the changes and updates!

@trusktr, it's weird about the sizing... we disable user zooming and let famo.us handle the sizing, so that's pretty weird. even on nexus 5 on the google emulator, it works fine. well, we'll track it down eventually i guess...

re performance, also pretty weird. i saw what you mean, but it's hard to reproduce. even with lagometer and some of my own code to log frame drops to the console, it only ever happens occasional. i think the code is ok because we only call setTransform(), and this shouldn't have any immediate performance impact and rather just animate according to Famo.us commits.

and yeah will add that icon when I have a chance.

gadicc avatar Nov 13 '14 11:11 gadicc

Seems like another good candidate for a plugin :smile:

You should have it tonight.

PEM-- avatar Nov 13 '14 11:11 PEM--