Test Test

bugs.jquery.com/ticket/9221
stackoverflow.com/questions/10903989/could-not-com...

Если нужно возвращать публичные методы лучше использовать такой вариант

(function( $ ){
	
	var methods = {
		
		init: function (options) {
                   var defaults = $.extend( {}, options ) //если передаете аргументы при инициализации
                },
                anotherMethod: function () {
                }
         }
         $.fn.pluginName = function (method) {
		
		if ( methods[method] ) {
				return methods[ method ].apply( this, Array.prototype.slice.call( arguments, 1 ));
		} else if ( typeof method === 'object' || ! method ) {
				return methods.init.apply( this, arguments );
		} else {
			  $.error( 'Метод ' +  method + ' не существует в jQuery.pluginName' );
		}
	}
})( jQuery );

Посмотрите, как люди пишут плагины.
Например, bootstrap tooltip.

// TOOLTIP PLUGIN DEFINITION
  // =========================

  function Plugin(option) {
    return this.each(function () {
      var $this   = $(this)
      var data    = $this.data('bs.tooltip')
      var options = typeof option == 'object' && option

      if (!data && /destroy|hide/.test(option)) return
      if (!data) $this.data('bs.tooltip', (data = new Tooltip(this, options)))
      if (typeof option == 'string') data[option]()
    })
  }

  var old = $.fn.tooltip

  $.fn.tooltip             = Plugin
  $.fn.tooltip.Constructor = Tooltip

[A-Z]\w*\.[a-z]\w*

По документации querySelector возвращает объект типа Element. Вы можете его расширить вашим плагином:

Element.prototype.myPlugin = function (params) {
  console.log(params.option);
};
document.querySelector('body').myPlugin({ option: true });

Запустите EXPLAIN запроса и посмотрите, на чём он тормозит. Из общих рекомендаций:
- cменить тип поля `products`.`archive` с текстового на BOOLEAN или TINYINT(1);
- проверить наличие индексов по все полям, участвующим в ON, WHERE, GROUP BY и ORDER BY.

Хотел бы спросить вашего мнения, господа!

Берем готовый фреймворк предоставляющий нам структуру - готово. Главное не делать лишних зависимостей и не пилить велосипеды. Low coupling и high cohesion.

Ну и да, если вы думаете о будущем но не пишите тестов - вы не думаете о будущем. Либо у вас или ваших QA слишком много свободного времени для поиска регрессий.

MVC (и развития этих идей: MVVM, MVP, MVA, PAC...)

Идея проста... между двумя слоями вводим третий который изолирует одно от другого. Слоеное лучше бисквита.

p.s. Если вы не знаете как это организовать, лучше на самом деле взять фреймворк или библиотеку, которая организует для вас слой представления и абстрагирует ваш код от него (React например).

Можно написать свой велосипед, но нужно ли?
requirejs.org/docs/whyamd.html

По ходу нашел решение на основе этого stackoverflow.com/questions/19076655/svg-sprite-ba...
В общем в background-size: необходимо указывать не только x но и y величину. На основе того, что у вашего оригинального svg спрайта (gamma.antikvarus.ru/themes/frontend_v2/images/svg/...) значения 960/560 (основываясь на viewBox), а на сайте у вас background-size: 424px; то в итоге получил такое значение
background-size: 424px 247px;
Как только его поставил, позиция svg стала одинаковой в хроме и фф, а если точнее, то стала как в ff, то есть смещенная вверх.
Тестировал на основе иконки логина (Вход), у которой по умолчанию было background-position: -65px -28px;
Вот так с измененными значениями (-22px вместо -28) выглядит в хроме

а так в FF


Так что просто добавляете y значение для background-size и увеличиваете y значение background-position на 6px.

iPad/iPhone предлагает какие-либо действия с телефоном/датой (позвонить, добавить в календарь итд), вероятно потому и выглядит как ссылка (дата даже во внутреннем месенджере превращается в ссылку с предложением добавить событие в календарь).
Пример того как можно бороться. Вообще ищется много вариантов, гуглите :)

Так будет красивее

function Mobile(newConfigure) {
  this.callbacks = []
  this.functions = {mobile: null, desktop: null};
}

Mobile.prototype.dispatcher = function(el, ev, dlg, cb) {
  this.callbacks.push({ el: el, ev: ev});  
  return $(el).on(ev, dlg, cb);
}

Mobile.prototype.removeListeners = function() {
  for (var i = 0, ln = this.callbacks.length; i < ln; i++ ) {
    $(this.callbacks[i].el).off(this.callbacks[i].ev);
  }
  this.callbacks = [];
}

Mobile.prototype.run = function(params) {
  //Здесь какое-либо условие 
  this.removeListeners();
  if (условие) {    
    params.mobile(this.dispatcher);
  } else {
    params.desktop(this.dispatcher);
  }
}

var mobile = new Mobile({
  mobile: function(on) {
    on('#myButton', 'click', function(e) { /* */ });
  },
  desktop: function(on) {
    on('#myButton', 'click', function(e) { /* */ });
  }
})

mobile.run();

А что значит: "... при переходе на другое состояние, он будет работать ..." ?
Что значит работать? У вас запускаются setInterval, setTimeout или вешаются события? Код выполняется один раз, он не может просто "работать"

git push с опцией --force, если никто больше не скачивал себе тот старый коммит и не пушил на сервер после Вас.
Либо можете сделать pull, потом push, но тогда на Bitbucket будет два почти одинаковых коммита.

До собеседования не приходилось тестовое делать.
Претенденту отправляем тестовое до собеседования только в случае, если по резюме нет уверенности, что стоит тратить время на этого человека

Для начала отмечу, что нет необходимости оборачивать заголовок в дополнительный тэг в вашем случае.
Рассмотрим варианты.

1. Заголовок и описание.

<header>
    <h2>Заголовок</h2>
    <p>Описание к заголовку</p>
</header>

Так как описание является продолжением заголовка в семантическом смысле, мы должны объединить их в одно смысловое целое, сохранив при этом разделение собственно заголовка и описания. Для этого используется тэг header, который, согласно спецификации, служит контейнером для создания шапки какого-либо элемента (не заголовка, а шапки).

2. Несколько заголовков.

<hgroup>
    <h1>Титульный заголовок</h1>
    <h2>Подзаголовок</h2>
</hgroup>

Так как у нашей секции несколько заголовков разного веса, их так же стоит семантически связать. Для этой цели можно использовать hgroup, который, согласно спецификации, служит для группирования нескольких заголовков. Но, так как его убрали из спецификации, полагаю, эту роль возьмет на себя header.

3. Ваш вариант.
<h2>Заголовок</h2>
Заметьте, у вас всего лишь один элемент, который не требует группировки с чем-либо. Он самодостаточен, и по смыслу соотносится с секцией, в которой находится. Нет никакого смысла выделять ему дополнительный контейнер в данном случае.

Hgroup:

Используется для группирования заголовков веб-страницы или раздела. Внутри располагаются теги заголовков от до . Данный тег исключён из версии HTML от W3C, но остался в версии WHATWG, будущее элемента пока не определено.

Header:

Тег задает «шапку» сайта или раздела, в которой обычно располагается заголовок.