Я знаю, что определенно поднимал вопрос по этому поводу в прошлом, но не могу вспомнить, как я его сформулировал, и было ли оно закрыто как невозможно воспроизвести или оставлено открытым для расследования 😬

Вы вообще можете это воспроизвести?

1013 была проблема, которую я поднял, почти то же самое, но не было открытого меню (не уверен, что это что-то меняет)

@rnystrom Может быть, стоит еще раз взглянуть, раз

Немногое обойтись без репро-банкомата. Если вы можете использовать FLEX и сбросить иерархию представлений, это наверняка даст нам начало.

_{Отправлено с помощью GitHawk}

Да, я могу воспроизвести, первое, что попробовал:

1. Открыть проблему со свернутым комментарием
2. Нажмите кнопку эмодзи, а затем быстро
3. Нажмите на комментарий над ним, чтобы развернуть

У меня есть видео, на котором я делаю это несколько раз, но я пока не могу его загрузить. Итак, вот еще одно изображение.

Вы можете сбросить иерархию представлений? Вы можете открыть FLEX , нажав тремя пальцами .

Конечно, я открыл Flex, но что мне делать дальше?

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

Вот один от меня

Приносим извинения за это, просто хочу посмотреть, может ли это случиться из-за большого количества базового текста!

// редактируем комментарии к этой супер-выполнимой функции, так что сужаем ее, поскольку она не вызвана альтернативными типами контента (например, изображениями)

Lorem ipsum dolor sit amet, conctetur adipiscing elit. Maecenas erat ipsum, Conctetuer Adipiscing nec mauris ac dolor eleifend orci. Donec aliquam sagittis mollis. Перед самым первым баскетбольным мячом их пасть горе и клиническая забота;

Для того, чтобы курс neque nisl, pulvinar urna Valley, и на футбол. Клинически в моих игроков. Я очень доволен элитой. Не сидеть долина, но пока правоохранительные органы на дуге. К сожалению, концы vel interdum vel Nulla neque aliquet, которые вызвали смех тяги.

Quisque nec ultricies sem. Зарезервировано, но стрелок нет, закончил снайпер, но осторожно горшок ес. Не делать, ни Suspendisse euismod sapien porttitor, erat volutpat felis lobortis. Футбол был ничем иным, как батареей.

и другой

Хорошо, много экспериментирую и редактирую предыдущий комментарий, я думаю, что сузил его до того, если вы расширите комментарий, и его новая высота выйдет за пределы экрана (плюс буфер).

Точно так же, как ячейка табличного представления загружается незадолго до того, как она появляется на экране, я подозреваю, что если новая высота выталкивает ячейку реакции из этого буфера, она не знает, где ее разместить? и так ничего не делает?

@rnystrom

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

Надеюсь, это имеет смысл, и я могу ошибаться, но, похоже, имеет смысл

Каждый может попробовать последнюю бета-версию. Вроде исправили?

_{Отправлено с помощью GitHawk}

Исправлено для меня. Отлично!

Все еще можно очень легко воспроизвести, выполнив те же действия, что и в исходном комментарии

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

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

Конечно, надо было сделать снимок экрана.

Поэтому я _sorta_ исправил это, отбросив здесь недействительный макет: https://github.com/rnystrom/GitHawk/blob/master/Classes/Issues/Comments/IssueCommentSectionController.swift#L137

collectionContext?.invalidateLayout(for: self, completion: nil)

Он не перемещает меню, но фактическая ячейка перемещается
🤔

Ага! Я вижу сейчас. Ok. Интересно, нужно ли нам просто что-то сделать, чтобы, если реакционная ячейка является первым реагирующим, мы не разворачивались? Я попробую, дружище.

изменить: нет, это не помогает. Хм. Я не знаком с тем, как меню работает внутри, нужно будет поиграться.

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

_{Отправлено с помощью GitHawk}

Я больше не могу воспроизводить

_{Отправлено с помощью GitHawk}

@gingerbeardman Даже с учетом новых шагов в моем предыдущем комментарии? (Нажатие на пузырь, затем на реакционную ячейку?)

Я могу довольно легко воспроизвести

Ах, хорошо, новые шаги.

Да, я могу воспроизвести.

Мои шаги сейчас:

1. Откройте панель смайликов
2. Коснитесь смайлика, чтобы добавить его в комментарий
3. Нажмите на комментарий, чтобы развернуть

@Sherlouk @gingerbeardman Я позволю вам, ребята, копать и находить здесь решение. Кажется, это довольно крайний случай, и вам нужно очень постараться, чтобы воспроизвести, поэтому не блокируйте 1.17.

_{Отправлено с помощью GitHawk}

@rnystrom Аннулирование макета, как я перечислил выше, всегда заставляло полосу реакции возвращаться туда, где она должна быть, и проблема была в меню. Меню больше не является проблемой, поэтому это должно быть окончательное решение - не уверены, влияет ли оно на производительность или что-то еще?

Ваше мнение?

С радостью оставлю это, если вы думаете, что это сверхнизкое качество изображения, но я испытал это пару раз, даже не пробуя ничего (в основном, когда есть изображение рядом с границей затухания)

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

1. Откройте панель смайликов
2. Коснитесь смайлика, чтобы добавить его в комментарий
3. Нажмите на комментарий, чтобы развернуть

Вот видео: http://www.youtube.com/watch?v=JJvny6hs0f8

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

@Sherlouk ya, если вы хотите выставить PR, аннулирующий там, мы можем дать этому шанс

_{Отправлено с помощью GitHawk}

Другой