Plots2: Соберите и исправьте последующие исправления Bootstrap 4

Привет @jywarren , вот обновленная суть обновленной карусели
https://gist.github.com/Souravirus/1f70b1ec63da1adaa686ccff7186ab2f

Кнопка основного изображения отлично отображается в firefox.

В браузере могут быть некоторые отличия. Видя это.

Хорошо, спасибо @Souravirus и спасибо @gauravano , я подумал, что нам следует начать новый 🙌

Я хотел бы внести свой вклад в этот выпуск! Спасибо!

@jywarren только что отредактировал выпуск: sweat_ smile :. Также добавлены наблюдения @stefannibrasil и @ sashadev-sky из # 5182. Спасибо!

Проблемы, о которых сообщил @stefannibrasil

ой ладно: v:

Первая страница выглядит странно. Может, кто-то снова подтолкнул другие коммиты?
Карусель не работает, и весь статический контент (значки социальных сетей, кнопка вопросов и т. Д.) Не отображается.

Некоторые места будут добавлены:

• [] между узлами и кнопками пагинации (первая страница);
• [] после кнопок типов узлов на странице поиска;
• [] между результатами узлов (кнопка «Читать дальше» прикреплена к изображению другого узла);

Главные примечания

• [] Любые несовпадения, наблюдаемые в переключателях или флажках, можно исправить с помощью классов в btsp_checkbox_override.css.scss . Обратите внимание, что эти несоответствия были до обновления, и обновление не устранило их --- by @ sashadev-sky. Подробнее см. Https://github.com/publiclab/plots2/issues/5182#issuecomment -481104366.
• [] Нижний колонтитул отличается при переходе между страницами. Например, на панели инструментов я вижу нижний колонтитул, отличный от того, который отображается на первой странице;
• [x] Также на панели инструментов, когда я нажимал «Поделиться своей работой», раскрывающийся список перемещается вправо, а не под ним;
• [] При добавлении вопроса в группу кнопок Markdown находится не на той же строке, что и другие;
• [] В выделенной заметке, которая отображается на панели инструментов, текст «Название узла пользователем1 из блога Public Lab» не выровнен;
• [] Заполнитель 'введите тег' здесь невозможно прочитать, например, https://unstable.publiclab.org/getting-started (возможно, удаление каретки решит эту проблему, поскольку уже видно, что это кнопка?)
• [x] Группа кнопок здесь не выровнена https://unstable.publiclab.org/wiki/plots-staff
• [x] Выровняйте кнопку на странице / subscriptions;
• [] Я думаю, что эти кнопки для вопросов и заметок об исследованиях слишком большие https://unstable.publiclab.org/wiki/organizers
• [x] Отображается только опция «выйти со всех устройств», не уверен, намеренно ли это;

Думаю, это все, хорошая работа! : тада:

Первая страница означает https://stable.publiclab.org/home : sweat_smile:

Я хотел бы внести свой вклад в этот выпуск! Спасибо!

Добро пожаловать, @ gautamig54 !!

@jywarren Проблема, о которой вы сообщили, с кнопкой ввода, происходит в Google Chrome

Привет, @jywarren , проблема с кнопкой ввода теперь решена. Вот скриншот:

Я предлагаю исправления в # 5613.

Отлично, спасибо !!!!

Пт, 26 апреля 2019 г., 14:35 Sourav Sahoo [email protected] написал:

Привет @jywarren https://github.com/jywarren , проблема с кнопкой ввода
теперь решено. Вот скриншот:
[image: image]
https://user-images.githubusercontent.com/17192149/56828907-fad66280-687f-11e9-93f6-e3b5897c4f6c.png
Я предлагаю исправления в # 5613
https://github.com/publiclab/plots2/pull/5613 .

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-487158505 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6J7X7CBA53E2KYVKIOTPSNDNPANCNFSM4HIH7DFA
.

Повторное открытие этой проблемы, так как некоторые ошибки не были решены.

Спасибо, @Souravirus - я пытаюсь вставить вашу суть (https://gist.github.com/Souravirus/1f70b1ec63da1adaa686ccff7186ab2f) для карусели на первой странице, а система функций выглядит немного странно; это также ошибки, когда я пытаюсь создать новую функцию:

Я знаю, что вы сдаетесь на экзамены, поэтому, если кто-то еще сможет помочь с этим, было бы здорово!

Похоже, что в форме довольно минимальные изменения:
https://github.com/publiclab/plots2/commit/7744d91e9bf03544da2f38e7e66056b6f20162e0#diff -d218cc0d572a2c87e465b2f68f3a20f8R5

Открою пиар на функциональную проверку на это!

Обновление: хм, уже есть функциональный тест. Попробую выложить более простой текст?

https://github.com/publiclab/plots2/blob/66aa809253e30781a1bdc8a974d67b0a47411687/test/functional/features_controller_test.rb#L76

Обновление: похоже, что это пост-сохранение в контроллере функций create . Если хотите воспроизвести в Codenvy, я открою новую проблему и перестану ее здесь обсуждать.

(Обновление: исправлено!)

(перемещено в отдельную проблему, поскольку в настоящее время это влияет на производственный сайт)

Привет @jywarren , я вижу, что карусель исправлена, но есть и другие проблемы, например:

• [x] Нет места после карусели и значков социальных сетей.
• [x] Карусель не меняется
• [x] После разделов «Проблема», «Совместная работа» и «Решение» ставится новая строка.

Да, я попробовал, но не до конца! Вот код:
https://gist.github.com/jywarren/3128766b5739555e6096aa12b27ab754

И я забыл, если вы администратор нестабильной версии, но я поместил это здесь:
https://stable.publiclab.org/features/14908/edit?t=1556900971

Я попробую еще раз в ближайшее время, мы должны решить эту проблему перед публикацией! Спасибо!

Вторник, 2 мая 2019 г., в 16:29 Sourav Sahoo [email protected]
написал:

Привет @jywarren https://github.com/jywarren , я вижу, что карусель исправлена
но есть и другие проблемы, например:

• Нет места после карусели и значков социальных сетей
• Карусель не меняется
• После проблемы, сотрудничества и символа новой строки
  разделы решения.

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-488790466 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6J5OZVH4KGKAN5UGA3DPTM2ONANCNFSM4HIH7DFA
.

Я не думаю, что я админ в нестабильной версии, так как не вижу ссылку.

Привет, @jywarren , работала на Slick , верно? Тогда почему это не сработало с Bootstrap 4. Что вы думаете по этому поводу?

Вы знаете, я думаю, что при необходимости мы можем заморозить карусель и просто показать
изображение в качестве краткосрочного исправления. Просто говорю, проще ли это! Я думал
Однако в этот момент остальная часть страницы находилась ниже. Я буду
посмотрите завтра !! Спасибо!!

Вс, 5 мая 2019 г., 16:17 Sourav Sahoo [email protected] написал:

Привет @jywarren https://github.com/jywarren , с предыдущей карусели
работал на пятне, верно? Тогда почему это не сработало с Bootstrap 4.
Что Вы думаете об этом?

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-489459810 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JZ3DI37LU5LLIFC4R3PT46FHANCNFSM4HIH7DFA
.

Да, я могу попробовать это исправить. Спасибо!!

классно!!!

В воскресенье, 5 мая 2019 г., в 16:34 Sourav Sahoo [email protected]
написал:

Да, я могу попробовать это исправить. Спасибо!!

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-489461145 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6J7E3DLQTPAYONPDMKLPT5AFFANCNFSM4HIH7DFA
.

@jywarren Я улучшил домашнюю страницу и сохранил только одно изображение вместо карусели.
Вот суть https://gist.github.com/Souravirus/7bb63f9188f955d85e856ea14e47caff
Спасибо!!

Выглядит отлично, Сурав, спасибо. Я могу немного поиграть с белым текстом, но он уже не идеален ...

https://stable.publiclab.org/home

Я замечаю, что теперь есть только 2 столбца, где мы раньше показывали 4;

нам нужно изменить стиль в views/notes/_notes.html.erb чтобы сделать его 4 столбцом?

Хм, это кажется подозрительным, дай мне посмотреть.

Привет @jywarren , на самом деле я допустил ошибку при обновлении. Я изменил col-md-3 col-lg-6, который был исправлен на col-lg-3. Итак, теперь все будет хорошо.
Спасибо!!

• [x] Нашел здесь пару маленьких; от одной функции мы, вероятно, избавимся, но она использует тег tabbed:notes для отображения вкладок; сравните эту страницу (https://stable.publiclab.org/wisconsin) со страницей PublicLab.org:

• [x] Другой маленький; я вижу это в нескольких местах, но группы кнопок иногда показывают разрывы, когда левая кнопка застревает в середине группы:

Вот ошибка, которую я получаю при входе на localhost. Итак, это ошибка @jywarren
Ошибка где-то в последней фиксации.

хм, ах! Гм, я только что объединил PR, который прошел тесты, посвященные этому. Сделал
что-то пошло не так со слиянием?

Во вторник, 7 мая 2019 г., в 16:05 Sourav Sahoo [email protected]
написал:

[image: image]
https://user-images.githubusercontent.com/17192149/57329335-34855400-7131-11e9-9382-ffc7f1e4542a.png
Вот ошибка, которую я получаю при входе на localhost. Так что это
ошибка @jywarren https://github.com/jywarren

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-490235028 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JZXTLKYT4FCYBCV7TTPUHOEVANCNFSM4HIH7DFA
.

Ха ... это должно быть доступно, не так ли?
https://github.com/publiclab/plots2/search?q=logged_in_as

Во вторник, 7 мая 2019 г., в 16:05 Sourav Sahoo [email protected]
написал:

[image: image]
https://user-images.githubusercontent.com/17192149/57329335-34855400-7131-11e9-9382-ffc7f1e4542a.png
Вот ошибка, которую я получаю при входе на localhost. Так что это
ошибка @jywarren https://github.com/jywarren

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-490235028 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JZXTLKYT4FCYBCV7TTPUHOEVANCNFSM4HIH7DFA
.

https://github.com/publiclab/plots2/blob/d55c716621d8072d0ffcd51b46c6b2b240ce5f4d/app/helpers/application_helper.rb#L3

Во вторник, 7 мая 2019 г., в 16:30 Джеффри Уоррен [email protected] написал:

Ха ... это должно быть доступно, не так ли?
https://github.com/publiclab/plots2/search?q=logged_in_as

Во вторник, 7 мая 2019 г., в 16:05 Sourav Sahoo [email protected]
написал:

[image: image]
https://user-images.githubusercontent.com/17192149/57329335-34855400-7131-11e9-9382-ffc7f1e4542a.png
Вот ошибка, которую я получаю при входе на localhost. Так что это
ошибка @jywarren https://github.com/jywarren

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-490235028 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JZXTLKYT4FCYBCV7TTPUHOEVANCNFSM4HIH7DFA
.

Подтвердите, помощники доступны для представлений, но не по умолчанию для контроллеров. Мы
нужно будет изменить, где мы это делаем с контроллеров ...

Во вторник, 7 мая 2019 г., в 16:30 Джеффри Уоррен [email protected] написал:

>

https://github.com/publiclab/plots2/blob/d55c716621d8072d0ffcd51b46c6b2b240ce5f4d/app/helpers/application_helper.rb#L3

Во вторник, 7 мая 2019 г., в 16:30 Джеффри Уоррен [email protected] написал:

Ха ... это должно быть доступно, не так ли?
https://github.com/publiclab/plots2/search?q=logged_in_as

Во вторник, 7 мая 2019 г., в 16:05 Sourav Sahoo [email protected]
написал:

[image: image]
https://user-images.githubusercontent.com/17192149/57329335-34855400-7131-11e9-9382-ffc7f1e4542a.png
Вот ошибка, которую я получаю при входе на localhost. Итак, это
это ошибка @jywarren https://github.com/jywarren

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-490235028 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JZXTLKYT4FCYBCV7TTPUHOEVANCNFSM4HIH7DFA
.

Хорошо, я сейчас скопирую этот метод в application_controller, просто
чтобы решить эту проблему.

Во вторник, 7 мая 2019 г., в 16:31 Джеффри Уоррен [email protected] написал:

Подтвердите, помощники доступны для представлений, но не по умолчанию для контроллеров. Мы
нужно будет изменить, где мы это делаем с контроллеров ...

Во вторник, 7 мая 2019 г., в 16:30 Джеффри Уоррен [email protected] написал:

>

https://github.com/publiclab/plots2/blob/d55c716621d8072d0ffcd51b46c6b2b240ce5f4d/app/helpers/application_helper.rb#L3

Во вторник, 7 мая 2019 г., в 16:30 Джеффри Уоррен [email protected] написал:

Ха ... это должно быть доступно, не так ли?
https://github.com/publiclab/plots2/search?q=logged_in_as

Во вторник, 7 мая 2019 г., в 16:05 Sourav Sahoo [email protected]
написал:

[image: image]
https://user-images.githubusercontent.com/17192149/57329335-34855400-7131-11e9-9382-ffc7f1e4542a.png
Вот ошибка, которую я получаю при входе на localhost. Итак, это
это ошибка @jywarren https://github.com/jywarren

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-490235028 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JZXTLKYT4FCYBCV7TTPUHOEVANCNFSM4HIH7DFA
.

но я хочу уловить это в тесте - и узнать, почему это не сработало!
https://github.com/publiclab/plots2/pull/5677

Во вторник, 7 мая 2019 г., в 16:39 Джеффри Уоррен

Хорошо, я сейчас скопирую этот метод в application_controller, просто
чтобы решить эту проблему.

Во вторник, 7 мая 2019 г., в 16:31 Джеффри Уоррен [email protected] написал:

Подтвердите, помощники доступны для представлений, но не по умолчанию для контроллеров. Мы
нужно будет изменить, где мы это делаем с контроллеров ...

Во вторник, 7 мая 2019 г., в 16:30 Джеффри Уоррен [email protected] написал:

>

https://github.com/publiclab/plots2/blob/d55c716621d8072d0ffcd51b46c6b2b240ce5f4d/app/helpers/application_helper.rb#L3

Во вторник, 7 мая 2019 г., в 16:30 Джеффри Уоррен [email protected]
написал:

Ха ... это должно быть доступно, не так ли?
https://github.com/publiclab/plots2/search?q=logged_in_as

Во вторник, 7 мая 2019 г., в 16:05 Sourav Sahoo [email protected]
написал:

[image: image]
https://user-images.githubusercontent.com/17192149/57329335-34855400-7131-11e9-9382-ffc7f1e4542a.png
Вот ошибка, которую я получаю при входе на localhost. Итак, это
это ошибка @jywarren https://github.com/jywarren

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-490235028 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JZXTLKYT4FCYBCV7TTPUHOEVANCNFSM4HIH7DFA
.

ОК, исправил! https://stable.publiclab.org/dashboard - до сих пор не уверен, как это не было обнаружено тестами ... может быть, это потому, что мы запускаем тесты с включенным ApplicationHelper? Попробую написать системный тест для дашборда!

Хорошо, тест системы приборной панели здесь: https://github.com/publiclab/plots2/pull/5682

• [Икс]
  @jywarren , на домашней странице эти элементы перекрываются, не могли бы вы прислать мне функции домашней страницы еще раз, я их
  И приятно слышать, что приборная панель работает нормально.
  :улыбка:

• [] Ах! Наша викторина с рассылкой спама немного веселая!

А вот и первая полоса! https://gist.github.com/jywarren/3128766b5739555e6096aa12b27ab754

Спасибо, @Souravirus !!!

Лол, я дал ответы 🐷 🐻 🐰 ✍️

Хм, я действительно вижу некоторые проблемы с форматированием всей формы регистрации:

@ IshaGupta18 @ CleverFool77 Я думаю, вы оба работали с этим файлом и, возможно, знакомы с ним. Кто-нибудь из вас хотел бы попробовать его рефакторинг, чтобы он хорошо работал в Bootstrap 4?

https://github.com/publiclab/plots2/blob/master/app/views/users/_create_form.html.erb

Потрясающая работа
Это ЭПИЧЕСКИЙ

@jywarren , эта кнопка на боковой панели, которая

Хм. в Bootstrap 3 был пример для этого сценария. Как насчет
это? https://getbootstrap.com/docs/4.3/components/button-group/#nesting

Вт, 9 мая 2019 г., в 15:47 Sourav Sahoo [email protected]
написал:

@jywarren https://github.com/jywarren , эта кнопка на боковой панели, которая
discontinues на самом деле является раскрывающимся списком, поэтому я не могу добавить его с другими
кнопки. И что я должен делать? Думаю, другие кнопки работают нормально, но
этот прекращается.

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-491040519 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6J6LIBOUQSSW25677KLPUR5TVANCNFSM4HIH7DFA
.

Хорошо, проверяю.

Вот обновленная домашняя страница для маленьких экранов https://gist.github.com/Souravirus/7bb63f9188f955d85e856ea14e47caff

Исправлены кнопки ОК

классно! Добавил в стабильную, работает отлично.

В четверг, 9 мая 2019 г., в 16:04 Sourav Sahoo [email protected]
написал:

Вот обновленная домашняя страница для маленьких экранов
https://gist.github.com/Souravirus/7bb63f9188f955d85e856ea14e47caff

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-491046033 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6J4FBYSBEW2PXPTIPJTPUR7TZANCNFSM4HIH7DFA
.

классно!!!

Вт, 9 мая 2019 г., в 16:28 Sourav Sahoo [email protected]
написал:

[image: image]
https://user-images.githubusercontent.com/17192149/57484448-aba02100-72c6-11e9-8acb-4aa44b6e5fda.png
Исправлены кнопки ОК

-
Вы получаете это, потому что вас упомянули.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612#issuecomment-491053985 ,
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JYVWTISPHUXWEHHRMDPUSCOJANCNFSM4HIH7DFA
.

просто добавив сюда еще несколько:

• [] изображение профиля в правом верхнем углу было круглым
• [x] перетаскивание для загрузки ведущей зоны изображения вики раньше было выше, если изображение еще не установлено - в основном в соответствии с верхней частью текстового поля
• [x] группы кнопок инструментов редактирования расположены слишком близко друг к другу

Также некоторые проблемы с заполнением на этих страницах:

• [x] Я думаю, что карточки "методов" требуют некоторого нижнего поля при ширине мобильного телефона:

Мелочь, но имеет значение!

Хорошо, и еще один здесь - нижний колонтитул, кажется, весь в одном столбце - мы почти у цели!

Исправлен нижний колонтитул! Это было в фиче!

Изображения профиля на странице вопросов кажутся завышенными:

https://publiclab.org/questions/stevie/05-20-2019/what-happens-to-the-simple-air-sensor-when-the-battery-pack-runs-low#answer -0-comment- 24212

И еще один небольшой, но важный момент - по мере того, как панель навигации становится все более узкой, мы настроили ее так, чтобы элементы появлялись в раскрывающемся меню, но теперь они не отображаются должным образом;

Ссылка на проблему https://github.com/publiclab/plots2/issues/5741 .

• [] обновление форм согласно bootstrap4

Всем привет, я хотел бы исправить некоторые последующие исправления для bootstrap4.
Спасибо !!!

Привет, мы будем рады твоей помощи! Особенно с навигационной панелью, собственно, как это
влияет на каждую страницу!

Во вторник, 21 мая 2019 г., в 7:52 Lekhika Dugtal [email protected]
написал:

Всем привет, я хотел бы внести несколько исправлений в bootstrap4.
Спасибо !!!

-
Вы получаете это, потому что изменили состояние открытия / закрытия.
Ответьте на это письмо напрямую, просмотрите его на GitHub
https://github.com/publiclab/plots2/issues/5612?email_source=notifications&email_token=AAAF6J7QFKKDWBONCODVABDPWPO57A5CNFSM4HIH7DFKYY3PNVWWWK3TUL52HS4DFMVREPNVWWK3TUL52HS3DFMVR08C02X3DFMVR08B08B08B08B0B08B0B08
или отключить поток
https://github.com/notifications/unsubscribe-auth/AAAF6JZCCFMFL5Z2HKWXA6DPWPO57ANCNFSM4HIH7DFA
.

Всем привет! спасибо за всю работу над этим. Ух ты!!

Одно предложение, которое я обнаружил с момента обновления, заключается в том, что я уменьшаю масштаб в окнах Public Lab до 75%, чтобы я мог видеть достаточно на одной странице, чтобы понять, что происходит. Кто-нибудь еще испытывает это? Есть ли способ уменьшить размер шрифта?

Привет, @ CleverFool77, действительно хорошая работа. Так держать!!