Sami Keijonen

Results 74 comments of Sami Keijonen

For example if in English you have long word, in Finnish it could be really short. And vice versa there is lot longer words in Finnish than in English. And...

I'm definitely gonna use `epäjärjestelmällistyttämättömyydellänsäkäänköhän` :)

> It's not uncommon for us to use custom navigation walkers to modify the structure or underlying output of a navigation menu to adhere to alternative FED methodologies (be it...

Noting that in the repo there are some issues opened what comes to accessibility.

> It probably needs some more testing but looks like my quick fix added RTL support for horizontal carousel slider setup, just check: [piotrku@a7b88d6](https://github.com/piotrku/tiny-slider/commit/a7b88d66e3513ebcd01df4bd062791271b1b3d24) @piotrku Thanks for this! In my...

Nice to see you here @atanas-angelov-dev! First, it's intentional that Justin has left out text-domain. It's added when you build your theme out of Mythic.

I keep PHPCS linting separate so that I can [ignore warnings in Travis](https://github.com/samikeijonen/uuups/blob/dev/.travis.yml#L47). [I do the same when pushing the code](https://github.com/samikeijonen/uuups/blob/master/package.json#L45). In other words, I also like to see those...

I'm not sure what you mean by visibility is not animatable property, it is if you ask me:) https://developer.mozilla.org/en/docs/Web/CSS/visibility Here is [demo](http://samikeijonen.github.io/responsive-nav.js/demos/multiple-levels/) (my own multi-level branch) where you can see...

We are not quite on the same page. First, we need `visibility: hidden/visible;` CSS for better keyboard navigation. `max-height` transition would not work smoothly when closing the menu if we...

Yes you can animate sub menus like that using bigger `max-height` value. I already tried that and use it also in couple of sites. But it's not perfect:) I think...