fomantic-ui 2.10.0-beta.1 → 2.10.0-beta.3

package/src/definitions/modules/search.js

@@ -134,7 +134,7 @@
                 determine: {
                     searchFields: function () {
                         // this makes sure $.extend does not add specified search fields to default fields
-                        // this is the only setting which should not extend defaults
+                        // this is the only setting that should not extend defaults
                         if (parameters && parameters.searchFields !== undefined) {
                             settings.searchFields = Array.isArray(parameters.searchFields)
                                 ? parameters.searchFields
@@ -660,7 +660,7 @@
                             ? searchFields
                             : settings.searchFields;
 
-                        // search fields should be array to loop correctly
+                        // search fields should be an array to loop correctly
                         if (!Array.isArray(searchFields)) {
                             searchFields = [searchFields];
                         }
@@ -938,7 +938,7 @@
                     id: function (results) {
                         module.debug('Injecting unique ids into results');
                         var
-                            // since results may be object, we must use counters
+                            // since results may be an object, we must use counters
                             categoryIndex = 0,
                             resultIndex   = 0
                         ;
@@ -1342,7 +1342,7 @@
         // minimum characters required to search
         minCharacters: 1,
 
-        // whether to select first result after searching automatically
+        // whether to select the first result after searching automatically
         selectFirstResult: false,
 
         // API config
@@ -1370,7 +1370,7 @@
         // Whether search result should highlight matching strings
         highlightMatches: false,
 
-        // match results also if they contain diacritics of the same base character (for example searching for "a" will also match "á" or "â" or "à", etc...)
+        // match results also if they contain diacritics of the same base character (for example, searching for "a" will also match "á" or "â" or "à", etc...)
         ignoreDiacritics: false,
 
         // whether to consider case sensitivity on local searching
@@ -1379,7 +1379,7 @@
         // whether to add events to prompt automatically
         automatic: true,
 
-        // delay before hiding menu after blur
+        // delay before hiding the menu after blur
         hideDelay: 0,
 
         // delay before searching

package/src/definitions/modules/shape.js

@@ -767,7 +767,7 @@
         onBeforeChange: function () {},
         onChange: function () {},
 
-        // allow animation to same side
+        // allow animation to the same side
         allowRepeats: false,
 
         // animation duration

package/src/definitions/modules/sidebar.js

@@ -389,7 +389,7 @@
                         if (module.othersActive()) {
                             module.debug('Other sidebars currently visible');
                             if (settings.exclusive) {
-                                // if not overlay queue animation after hide
+                                // if not overlay, queue animation after hide
                                 if (settings.transition !== 'overlay') {
                                     module.hideOthers(module.show);
 

package/src/definitions/modules/slider.js

@@ -271,7 +271,7 @@
                             isHover = false;
                         });
                         // All touch events are invoked on the element where the touch *started*. Thus, we can bind them all
-                        // on the thumb(s) and don't need to worry about interference with other components, i.e. no dynamic binding
+                        // on the thumb(s) and don't need to worry about interference with other components, i.e., no dynamic binding
                         // and unbinding required.
                         $module.find('.thumb')
                             .on('touchstart' + eventNamespace, module.event.touchDown)
@@ -281,7 +281,7 @@
                         ;
                     },
                     slidingEvents: function () {
-                        // these don't need the identifier because we only ever want one of them to be registered with document
+                        // these don't need the identifier because we only ever want one of them to be registered with the document
                         module.verbose('Binding page wide events while handle is being draged');
                         $document.on('mousemove' + eventNamespace, module.event.move);
                         $document.on('mouseup' + eventNamespace, module.event.up);
@@ -506,7 +506,7 @@
                 },
 
                 resync: function () {
-                    module.verbose('Resyncing thumb position based on value');
+                    module.verbose('Re-syncing thumb position based on value');
                     module.setup.labels();
                     if (module.is.range()) {
                         module.update.position(module.secondThumbVal, $secondThumb);
@@ -985,7 +985,7 @@
                         if (value !== difference) {
                             module.verbose('Rounding value to closest step: ' + difference);
                         }
-                        // Use precision to avoid ugly Javascript floating point rounding issues
+                        // Use precision to avoid ugly JavaScript floating point rounding issues
                         // (like 35 * .01 = 0.35000000000000003)
                         module.verbose('Cutting off additional decimal places');
 

package/src/definitions/modules/slider.less

@@ -106,7 +106,7 @@
     --------------- */
 
     .ui.disabled.slider@{notCheckbox} {
-        opacity: @disabledOpactiy;
+        opacity: @disabledOpacity;
     }
 
     .ui.disabled.slider .inner:hover {

package/src/definitions/modules/sticky.js

@@ -871,10 +871,10 @@
         // Offset to adjust scroll when attached to bottom of screen
         bottomOffset: 0,
 
-        // will only set container height if difference between context and container is larger than this number
+        // will only set container height if the difference between context and container is larger than this number
         jitter: 5,
 
-        // set width of sticky element when it is fixed to page (used to make sure 100% width is maintained if no fixed size set)
+        // set width of the sticky element when it is fixed to page (used to make sure 100% width is maintained if no fixed size set)
         setSize: true,
 
         // Whether to automatically observe changes with Mutation Observers
@@ -886,10 +886,10 @@
         // Called on each scroll
         onScroll: function () {},
 
-        // Called when element is stuck to viewport
+        // Called when the element is stuck to viewport
         onStick: function () {},
 
-        // Called when element is unstuck from viewport
+        // Called when the element is unstuck from viewport
         onUnstick: function () {},
 
         // Called when element reaches top of context

package/src/definitions/modules/tab.js

@@ -320,7 +320,7 @@
                         pushStateAvailable = window.history && window.history.pushState,
                         shouldIgnoreLoad   = pushStateAvailable && settings.ignoreFirstLoad && firstLoad,
                         remoteContent      = settings.auto || $.isPlainObject(settings.apiSettings),
-                        // only add default path if not remote content
+                        // only add the default path if not remote content
                         pathArray = remoteContent && !shouldIgnoreLoad
                             ? module.utilities.pathToArray(tabPath)
                             : module.get.defaultPathArray(tabPath)
@@ -392,7 +392,7 @@
                             $anchor = $('#' + tabPath + ', a[name="' + tabPath + '"]');
                             currentPath = $anchor.closest('[data-tab]').data(metadata.tab);
                             $tab = module.get.tabElement(currentPath);
-                            // if anchor exists use parent tab
+                            // if anchor exists, use parent tab
                             if ($anchor && $anchor.length > 0 && currentPath) {
                                 module.debug('Anchor link used, opening parent tab', $tab, $anchor);
                                 if (settings.onBeforeChange.call(element, currentPath) === false) {
@@ -471,9 +471,6 @@
                                 encodeParameters: false,
                                 on: 'now',
                                 cache: settings.alwaysRefresh,
-                                headers: {
-                                    'X-Remote': true,
-                                },
                                 onSuccess: function (response) {
                                     if (settings.cacheType === 'response') {
                                         module.cache.add(fullTabPath, response);
@@ -618,7 +615,7 @@
                     path: function () {
                         return $.address.value();
                     },
-                    // adds default tabs to tab path
+                    // adds default tabs to the tab path
                     defaultPathArray: function (tabPath) {
                         return module.utilities.pathToArray(module.get.defaultPath(tabPath));
                     },
@@ -902,7 +899,7 @@
         verbose: false,
         performance: true,
 
-        auto: false, // uses pjax style endpoints fetching content from same url with remote-content headers
+        auto: false, // uses pjax style endpoints fetching content from the same url with remote-content headers
         history: false, // use browser history
         historyType: 'hash', // #/ or html5 state
         path: false, // base path of url
@@ -916,21 +913,21 @@
         alwaysRefresh: false, // load tab content new every tab click
         cache: true, // cache the content requests to pull locally
         loadOnce: false, // Whether tab data should only be loaded once when using remote content
-        cacheType: 'response', // Whether to cache exact response, or to html cache contents after scripts execute
-        ignoreFirstLoad: false, // don't load remote content on first load
+        cacheType: 'response', // Whether to cache exact response, or to HTML cache contents after scripts execute
+        ignoreFirstLoad: false, // don't load remote content on the first load
 
         apiSettings: false, // settings for api call
         evaluateScripts: 'once', // whether inline scripts should be parsed (true/false/once). Once will not re-evaluate on cached content
-        autoTabActivation: true, // whether a non existing active tab will auto activate the first available tab
+        autoTabActivation: true, // whether a non-existing active tab will auto activate the first available tab
 
         onFirstLoad: function (tabPath, parameterArray, historyEvent) {}, // called first time loaded
         onLoad: function (tabPath, parameterArray, historyEvent) {}, // called on every load
         onVisible: function (tabPath, parameterArray, historyEvent) {}, // called every time tab visible
-        onRequest: function (tabPath, parameterArray, historyEvent) {}, // called ever time a tab beings loading remote content
+        onRequest: function (tabPath, parameterArray, historyEvent) {}, // called every time a tab beings loading remote content
         onBeforeChange: function (tabPath) {}, // called before a tab is about to be changed. Returning false will cancel the tab change
 
         templates: {
-            determineTitle: function (tabArray) {}, // returns page title for path
+            determineTitle: function (tabArray) {}, // returns page title for the path
         },
 
         error: {

package/src/definitions/modules/toast.js

@@ -826,8 +826,8 @@
 
         title: '',
         message: '',
-        displayTime: 3000, // set to zero to require manually dismissal, otherwise hides on its own
-        minDisplayTime: 1000, // minimum displaytime in case displayTime is set to 'auto'
+        displayTime: 3000, // set to zero to require manual dismissal, otherwise hides on its own
+        minDisplayTime: 1000, // minimum display time in case displayTime is set to 'auto'
         wordsPerMinute: 120,
         showIcon: false,
         newestOnTop: false,
@@ -850,7 +850,7 @@
             showDuration: 500,
             hideMethod: 'scale',
             hideDuration: 500,
-            closeEasing: 'easeOutCubic', // Set to empty string to stack the closed toast area immediately (old behaviour)
+            closeEasing: 'easeOutCubic', // Set to empty string to stack the closed toast area immediately (old behavior)
             closeDuration: 500,
         },
 

package/src/definitions/modules/transition.js

@@ -588,7 +588,7 @@
                         if (shouldDetermine && $module.data(metadata.displayType) === undefined) {
                             var currentDisplay = $module.css('display');
                             if (currentDisplay === '' || currentDisplay === 'none') {
-                                // create fake element to determine display state
+                                // create a fake element to determine display state
                                 module.can.transition(true);
                             } else {
                                 module.save.displayType(currentDisplay);
@@ -967,7 +967,7 @@
         // event namespace
         namespace: 'transition',
 
-        // delay between animations in group
+        // delay between animations in a group
         interval: 0,
 
         // whether group animations should be reversed
@@ -990,13 +990,13 @@
         // whether timeout should be used to ensure callback fires in cases animationend does not
         useFailSafe: true,
 
-        // delay in ms for fail safe
+        // delay in ms for fail-safe
         failSafeDelay: 100,
 
         // whether EXACT animation can occur twice in a row
         allowRepeats: false,
 
-        // Override final display type on visible
+        // Override the final display type on visible
         displayType: false,
 
         // animation duration

package/src/themes/default/modules/slider.variables

@@ -46,7 +46,7 @@
 -------------------- */
 
 /* Disabled */
-@disabledOpactiy: 0.5;
+@disabledOpacity: 0.5;
 @disabledTrackFillColor: @background;
 
 /* -------------------