Bug 55117

Summary: phosh: Исправление ошибки с тем, что клавиатуру на экране приложений нельзя спрятать
Product: Sisyphus Reporter: Vladimir Romanov <rirusha>
Component: phoshAssignee: Yuri N. Sedunov <aris>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P5 CC: aris, bircoph, oleg, rirusha
Version: unstable   
Hardware: all   
OS: Linux   
Attachments:
Description Flags
Сам патч
none
Патч, который позволяет прятать экранную клавиатуру после начала поиска none

Description Vladimir Romanov 2025-07-09 15:11:25 MSK
Created attachment 19010 [details]
Сам патч

Апстрим отказался принимать патч в виду того, что у них своё решение исправления проблемы (Stevia + исправление бага с непоказом по клику на зафокушеном поле), но... ALT Mobile пока на squeekboard, да и со stevia мы заменим одни проблемы другими.

Прикладываю патч
Comment 1 Yuri N. Sedunov 2025-07-09 15:16:17 MSK
(Ответ для Vladimir Vaskov на комментарий #0)
> Создано вложение 19010 [details] [подробности]
> Сам патч
> 
> Апстрим отказался принимать патч 
Ссылку давай, да.
Comment 2 Vladimir Romanov 2025-07-09 17:55:19 MSK
https://gitlab.gnome.org/World/Phosh/phosh/-/merge_requests/1744

Как пожелаете
Comment 3 Yuri N. Sedunov 2025-07-09 18:01:57 MSK
Апстрим предлагает не трогать phosh и зафиксить squeekboard.
Comment 4 Vladimir Romanov 2025-07-09 18:11:15 MSK
Предложенное "исправление" создаст баги с тем, что клавиатура в меню приложений работает не от состояния клавиатуры, а от фокуса на поле поиска. Если просто спрятать клавиатуру, то любое переключение фокуса, а затем его возврат, покажет клавиатуру. Да и появятся проблемы, как у stevia. Убрать то убрали, а показать не можем. Это уже другой баг...
Comment 5 Andrew Savchenko 2025-07-09 18:12:38 MSK
(Ответ для Yuri N. Sedunov на комментарий #3)
> Апстрим предлагает не трогать phosh и зафиксить squeekboard.

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

А патч rirusha позволяет решить проблему вне зависимости от клавиатуры.
Comment 6 Yuri N. Sedunov 2025-07-09 18:27:24 MSK
(Ответ для Vladimir Vaskov на комментарий #4)
> Убрать то убрали, а показать не можем. Это уже другой баг...

Этот "другой" баг есть в апсриме?

(Ответ для Andrew Savchenko на комментарий #5)
> (Ответ для Yuri N. Sedunov на комментарий #3)
> > Апстрим предлагает не трогать phosh и зафиксить squeekboard.
> 
> Мы можем перейти на Stevia, но тогда баг с невозможностью закрыть клавиатуру
> заменится на баг с невозможностью её открыть после закрытия, что не сильно
> лучше.
> 
> А патч rirusha позволяет решить проблему вне зависимости от клавиатуры.

Доверие к рирушиным патчам так себе после
https://bugzilla.altlinux.org/52630
https://gitlab.gnome.org/World/Phosh/phosh/-/merge_requests/1583

Продвигайте свои решения в апстрим.
Comment 7 Andrew Savchenko 2025-07-10 23:58:50 MSK
(Ответ для Yuri N. Sedunov на комментарий #6)
> (Ответ для Vladimir Vaskov на комментарий #4)
> > Убрать то убрали, а показать не можем. Это уже другой баг...
> 
> Этот "другой" баг есть в апсриме?

Да, есть:
https://gitlab.gnome.org/World/Phosh/phosh/-/issues/1196
 
> (Ответ для Andrew Savchenko на комментарий #5)
> > (Ответ для Yuri N. Sedunov на комментарий #3)
> > > Апстрим предлагает не трогать phosh и зафиксить squeekboard.
> > 
> > Мы можем перейти на Stevia, но тогда баг с невозможностью закрыть клавиатуру
> > заменится на баг с невозможностью её открыть после закрытия, что не сильно
> > лучше.
> > 
> > А патч rirusha позволяет решить проблему вне зависимости от клавиатуры.
> 
> Доверие к рирушиным патчам так себе после
> https://bugzilla.altlinux.org/52630

Ради избежания повторения столь неприятной ситуации я и попросил Рирушу отправить патч в апстрим и Вам.

> https://gitlab.gnome.org/World/Phosh/phosh/-/merge_requests/1583

Эта история как раз закончилась хорошо, в т.ч. благодаря Володе: хотя апстрим пошёл иным путём и его патч не принял, но главное, чтоб проблема решена:
https://gitlab.gnome.org/World/Phosh/phosh/-/issues/1084#note_2410531

> Продвигайте свои решения в апстрим.

У нас есть два варианта: текущий патч или патчить squeekboard. Но апстрим squeekboard более не развивает, так что там наиболее вероятный сценарий, что патч снова придётся продвигать в Сизиф после отсутствия интереса со стороны апстрима.

Со временем мы перейдём на Stevia, но решение проблемы нужно раньше :/
Comment 8 Yuri N. Sedunov 2025-07-11 07:54:51 MSK
(Ответ для Andrew Savchenko на комментарий #7)
> (Ответ для Yuri N. Sedunov на комментарий #6)
> > (Ответ для Vladimir Vaskov на комментарий #4)
> > > Убрать то убрали, а показать не можем. Это уже другой баг...
> > 
> > Этот "другой" баг есть в апсриме?
> 
> Да, есть:
> https://gitlab.gnome.org/World/Phosh/phosh/-/issues/1196
>  

Дополните message подробным описанием проблемы с упоминанием squeekboard, со ссылками на этот баг и непринятый mr. Назовите патч по нашим правилам.
Comment 9 Vladimir Romanov 2025-07-11 18:05:51 MSK
Created attachment 19079 [details]
Патч, который позволяет прятать экранную клавиатуру после начала поиска

В phosh есть проблема: невозможность спрятать экранную клавиатуру во время поиска и переименовывания папки. Если во втором есть кнопки принятия и отмены, которые могут "взять" фокус и спрятать клавиатуру, то у поиска такого нет. Добавлена кнопка принтия поиска и поведение с кнопкой enter, чтобы клавиатуру после поиска можно было убрать.

Апстрим от патча отказался (https://gitlab.gnome.org/World/Phosh/phosh/-/merge_requests/1744). Там своё видение, что клавиатура должна уметь прятаться сама. Но даже с иным провайдером экранной клавиатуры (например stevia), есть проблема с тем, что показать клавиатуру нельзя: если поле ввода уже содержит фокус, то клавиатура не покажется (https://gitlab.gnome.org/World/Phosh/phosh/-/issues/1196).

Но даже если этот баг будет исправлен, появятся не столько баги, сколько неудобства в UX: Если начать поиск, убрать клавиатуру, затем переместиться на окно приложения, а затем вернуться, то клавиатура появится. Логика экранной клавиатуры привязана к фокусу на поле поиска.
Comment 10 Repository Robot 2025-07-11 19:34:44 MSK
phosh-0.48.0-alt1.1 -> sisyphus:

Fri Jul 11 2025 Yuri N. Sedunov <aris@altlinux> 0.48.0-alt1.1
- applied experimental patch proposed in (ALT #55117)