On Thu, Nov 20, 2014 at 12:08 PM, Shahyar Ghobadpour <sghobadpour@wikimedia.org> wrote:--I missed standup, my apologies. TOC is up and running on flow-tests. Can be product reviewed, look for any browser bugs. Non show-stoppers for now:- I've noticed a couple of minor issues in Firefox, but not in Chrome, so I'm trying to isolate those. Need to test other browsers as well.- I need to implement the new loader animation (ellipsis).It's bounce[123] in showLoadingIndicator() in modules/source/ext.cx.source.js.
We and ContentTranslation (git blame says Santhosh Thottingal worked on it) should propose this for MW core, https://trello.com/c/2u8W6qn8- Gotta remove the "no older topics" message from TOC.I guess so. When we have a consistent loading animation at the bottom of the TOC I think it'll be obvious when you've reached the end.(We don't have a spec for the no-JS TOC.)
Danny and I noticed that the TOC doesn't always load the rest of the titles. It got stuck for me at 60 items.
(The numbering in the TOC is very helpful, I assume it's temporary along with loading 10 at a time instead of 100.)- Need to add "X" to close TOC (I only noticed this in the requirements now -- it does not exist on any of the design mockups except for ONE).Which mockup? I haven't missed it. The Echo flyout doesn't have a close "X". Danny Pau what do you think?- Code needs to be split up, but this doesn't prevent product reviewing.=S Page Features engineer