-
Notifications
You must be signed in to change notification settings - Fork 148
Предотвращать проскролл вверх по клику на иконку копирования пути теста #678
Предотвращать проскролл вверх по клику на иконку копирования пути теста #678
Conversation
|
в gemini-gui локально проверил, работает, |
lib/reporters/html/static/report.js
Outdated
| 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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
можно комментарий не оставлять, вроде и так понятно зачем это
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
если настаиваешь, уберу весь коммент, но не кажется что те, кто не в контексте этой проблемы не поймут зачем это
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
о, просто оставлю ссылку в этом PR, по блэйму найдётся..
lib/reporters/html/static/report.js
Outdated
|
|
||
| function handleClipboard() { | ||
| forEach.call(document.querySelectorAll('.section__icon_copy-to-clipboard'), function(clipboard) { | ||
| /* eslint-disable no-new */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
перенеси, плз, этот комментарий ниже - он для строки 182
для проверки в gemini нужно выполнить команду |
а при чем здесь gemini-gui? это же отдельный инструмент - у них разные репорты. Просто по функциональности и визуально они похожи :) |
|
собрал gemini с твоим фиксом - в iframe-е все равно в топ скролится. А локально у меня и без фикса работает правильно |
|
Ребят, привет! Скажите, а у вас в итоге получилось обойти этот баг? В каких случаях ваш патч помогает, а в каких нет? |
|
я продолжаю мучаться с багом, тут я не знаю что делать |
|
Пофикшено в |
zenorocha/clipboard.js#320 (comment)