Skip to content
This repository was archived by the owner on Sep 21, 2022. It is now read-only.

Conversation

@a-x-
Copy link

@a-x- a-x- commented Nov 22, 2016

@a-x-
Copy link
Author

a-x- commented Nov 22, 2016

в gemini-gui локально проверил, работает,
тесты писать сейчас нет времени :/

function handleClipboard() {
forEach.call(document.querySelectorAll('.section__icon_copy-to-clipboard'), function(clipboard) {
/* eslint-disable no-new */
// Не прокручивать страницу вверх - github.com/zenorocha/clipboard.js/issues/320#issuecomment-262261154
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Author

@a-x- a-x- Nov 23, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

если настаиваешь, уберу весь коммент, но не кажется что те, кто не в контексте этой проблемы не поймут зачем это

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

о, просто оставлю ссылку в этом PR, по блэйму найдётся..


function handleClipboard() {
forEach.call(document.querySelectorAll('.section__icon_copy-to-clipboard'), function(clipboard) {
/* eslint-disable no-new */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

перенеси, плз, этот комментарий ниже - он для строки 182

@sipayRT
Copy link
Member

sipayRT commented Nov 23, 2016

как проверить пока не знаю

для проверки в gemini нужно выполнить команду npm run prepublish, чтобы перегенерить клиентский скрипт, а потом запустить много тестов, чтобы построился большой отчет. Я проверю

@sipayRT
Copy link
Member

sipayRT commented Nov 23, 2016

в gemini-gui локально проверил, работает

а при чем здесь gemini-gui? это же отдельный инструмент - у них разные репорты. Просто по функциональности и визуально они похожи :)

@sipayRT
Copy link
Member

sipayRT commented Nov 24, 2016

собрал gemini с твоим фиксом - в iframe-е все равно в топ скролится. А локально у меня и без фикса работает правильно

@kachkaev
Copy link

kachkaev commented Jan 4, 2017

Ребят, привет! Скажите, а у вас в итоге получилось обойти этот баг? В каких случаях ваш патч помогает, а в каких нет?

@a-x-
Copy link
Author

a-x- commented Jan 18, 2017

я продолжаю мучаться с багом,
у меня фикс работает, но @sipayRT говорит, что у него всё ок, а фикс ему ломает...

тут я не знаю что делать

@j0tunn
Copy link
Contributor

j0tunn commented May 12, 2017

Пофикшено в [email protected]

@j0tunn j0tunn closed this May 12, 2017
@j0tunn j0tunn removed the review label May 12, 2017
@a-x- a-x- deleted the inv.prevent-copy-to-clipboard-scroll branch May 12, 2017 16:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants