5 commits

Author	SHA1	Message	Date
Stephen Niedzielski	a4e129175a	Fix: show thumbnails on left for right-to-left UIs ... When the UI is RTL, show preview thumbnails on the left. Otherwise, show them on the right. Bug: T190831 Change-Id: Ic1fc54f6547b31908905db8cb2ec4d58f37a6538	2018-04-23 16:59:19 -05:00
Stephen Niedzielski	44a7f643bc	Hygiene: replace CSS class underscores w/ hyphens ... Replace CSS slithery_snake_case with shish-kebab-case for consistency with the rest of the codebase. find \ -not '( ( -name node_modules -o -name .git -o -name vendor -o -name doc -o -name dist ) -prune )' \ -type f| xargs \ -rd\\n \ sed -ri 's%flipped_([xy])_([xy])%flipped-\1-\2%g; s%flipped_([xy])%flipped-\1%g' Change-Id: I25dc0ddeda711faca9a79b5bf87d6b5aa0d5aea5	2018-04-23 16:23:17 -05:00
Stephen Niedzielski	3a372ac3ab	Hygiene: rename triangle terminology to pokey ... Everyone knows what a poke is. find \ -not '( ( -name node_modules -o -name .git -o -name vendor -o -name doc -o -name dist ) -prune )' -type f| xargs -rd\\n sed -ri 's%\btri(\b|angle)%pokey%g' Change-Id: Ie159aa6947801a98cbf358da1613c87cf66d548f	2018-04-23 16:07:51 -05:00
Stephen Niedzielski	2eeaa0a2e4	Hygiene: consolidate clip-path manipulation ... The clip-path SVG property was conditionally set in thumbnail.js and also conditionally set or removed in renderer.js. This patch refactors the logic to occur in a single place, renderer.js. The refactor was made with the following considerations: • The one condition under which thumbnail.js would set clip-path was, given a thumbnail exists, the thumbnail was not tall and clip-path would be set to `url(#mwe-popups-mask)`. Otherwise, thumbnail.js would not set clip-path. • The logic in renderer.js for setting the attribute doesn't change since overwriting the clip-path is equivalent to not having a preexisting value. The case for removing the attribute itself is replaced by inverting the condition, `flippedY`, and combining it with the thumbnail.js condition, `!isTall`. The operation is only valid for an existing thumbnail so the `hasThumbnail` remains unchanged. This patch also clarifies that the "flipped" classes are exclusively set by using an if / else chain instead of reconsidering all inputs for each condition. Bug: T190831 Change-Id: I4062ec7068dcadecbdbc4791447ea2ed1ce2a1de	2018-04-13 09:25:22 -05:00
Stephen Niedzielski	c61513f0ea	Hygiene: split LESS into components ... Bug: T165036 Change-Id: I916f27138b5abe6b9fe23719cffb10f20e604b89	2018-04-03 16:37:05 -07:00

Renamed from resources/ext.popups.main/styles/ext.popups.core.less (Browse further)