New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Усовершенствовать вызов контекстного меню #672

Closed
awd-studio opened this Issue Sep 23, 2017 · 18 comments

Comments

Projects
None yet
5 participants
@awd-studio
Contributor

awd-studio commented Sep 23, 2017

В новом дизе, есть фича, блокирующая контекстное меню браузера по ПКМ на комментах. Оно, вроде как, круто... Но, если честно, я - противник подобных фич. И нахожу это жутко неудобным.

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

Лично, использую контекстное меню очень часто - от открытия картинок в новом табе, до инспектирования узла в дэвтулзе.

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

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Sep 23, 2017

Contributor

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

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

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

Contributor

mbaev commented Sep 23, 2017

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

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

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

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Sep 23, 2017

Contributor

Лично я вообще только так и отвечаю на комменты.

Contributor

mbaev commented Sep 23, 2017

Лично я вообще только так и отвечаю на комменты.

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Sep 23, 2017

Contributor

Отключать полностью - думаю, точно не стоит. А вот доделать - да. Ну, хотябы чтоб с фокса работало полноценно. И про настройку в профиле, особенно для тех кому не нравится не стандартное поведение, думаю есть смысл подумать (я даже реализовать мог бы).

ЗЫ - У меня когда-то была идея сделать хоткеи на разные действия, например для цитаты можно было бы выделить текст и жмакнуть Q (гиттер-стайл). Но, чота отсохло...

Contributor

awd-studio commented Sep 23, 2017

Отключать полностью - думаю, точно не стоит. А вот доделать - да. Ну, хотябы чтоб с фокса работало полноценно. И про настройку в профиле, особенно для тех кому не нравится не стандартное поведение, думаю есть смысл подумать (я даже реализовать мог бы).

ЗЫ - У меня когда-то была идея сделать хоткеи на разные действия, например для цитаты можно было бы выделить текст и жмакнуть Q (гиттер-стайл). Но, чота отсохло...

@Director-cemetery

This comment has been minimized.

Show comment
Hide comment
@Director-cemetery

Director-cemetery Sep 23, 2017

например для цитаты можно было бы выделить текст и жмакнуть Q (гиттер-стайл).

Мощняво было бы.

Director-cemetery commented Sep 23, 2017

например для цитаты можно было бы выделить текст и жмакнуть Q (гиттер-стайл).

Мощняво было бы.

@itcrowd72

This comment has been minimized.

Show comment
Hide comment
@itcrowd72

itcrowd72 Sep 23, 2017

Member

Плюсую @awd-studio
Фича в FF не работает как положено и немного раздражает

Member

itcrowd72 commented Sep 23, 2017

Плюсую @awd-studio
Фича в FF не работает как положено и немного раздражает

@mbaev mbaev added the UI/UX label Oct 2, 2017

@mbaev mbaev assigned mbaev and adubovskoy and unassigned mbaev Oct 29, 2017

@mbaev mbaev added this to the 6-ое ноября 2017 milestone Oct 29, 2017

@mbaev mbaev added the Improvement label Oct 29, 2017

mbaev added a commit that referenced this issue Oct 29, 2017

@itcrowd72

This comment has been minimized.

Show comment
Hide comment
@itcrowd72

itcrowd72 Oct 31, 2017

Member

Коммит есть. issue забыли закрыть?

Member

itcrowd72 commented Oct 31, 2017

Коммит есть. issue забыли закрыть?

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 31, 2017

Contributor

Да, походу.

Contributor

mbaev commented Oct 31, 2017

Да, походу.

@mbaev mbaev closed this Oct 31, 2017

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Oct 31, 2017

Contributor

Аааа. Хочу еще обсудить )))
Вот на YouTube классно реализована фича, давайте и у нас запилим?

Там при первом ПКМ открывается их контекстка, а при втором - нормальная.
ИМХО, решает проблему.

Contributor

awd-studio commented Oct 31, 2017

Аааа. Хочу еще обсудить )))
Вот на YouTube классно реализована фича, давайте и у нас запилим?

Там при первом ПКМ открывается их контекстка, а при втором - нормальная.
ИМХО, решает проблему.

@itcrowd72

This comment has been minimized.

Show comment
Hide comment
@itcrowd72

itcrowd72 Oct 31, 2017

Member

Лично я против переопределения сайтами контекстного меню) Хотя вариант YouTube имеет место быть

Member

itcrowd72 commented Oct 31, 2017

Лично я против переопределения сайтами контекстного меню) Хотя вариант YouTube имеет место быть

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 31, 2017

Contributor

Это уже другой тред, создайте отдельную задачу. Эта задача, в её контексте завершена.

Contributor

mbaev commented Oct 31, 2017

Это уже другой тред, создайте отдельную задачу. Эта задача, в её контексте завершена.

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Oct 31, 2017

Contributor

Ссылки теперь не работают при вызове контекстного меню правым кликом

Contributor

mbaev commented Oct 31, 2017

Ссылки теперь не работают при вызове контекстного меню правым кликом

@mbaev mbaev reopened this Oct 31, 2017

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Nov 1, 2017

Contributor

@adubovskoy глянешь?

Contributor

mbaev commented Nov 1, 2017

@adubovskoy глянешь?

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Nov 3, 2017

Contributor

@awd-studio или кто-то ещё может посмотреть в чём дело и исправить баг?

Contributor

mbaev commented Nov 3, 2017

@awd-studio или кто-то ещё может посмотреть в чём дело и исправить баг?

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Nov 3, 2017

Contributor

Ок, я гляну сегодня-завтра.

Contributor

awd-studio commented Nov 3, 2017

Ок, я гляну сегодня-завтра.

awd-studio added a commit to awd-studio/drupal.ru that referenced this issue Nov 4, 2017

awd-studio added a commit to awd-studio/drupal.ru that referenced this issue Nov 4, 2017

@awd-studio awd-studio self-assigned this Nov 4, 2017

awd-studio added a commit to awd-studio/drupal.ru that referenced this issue Nov 4, 2017

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Nov 4, 2017

Contributor

Вот на YouTube классно реализована фича, давайте и у нас запилим?

Там при первом ПКМ открывается их контекстка, а при втором - нормальная.

Реализовал в #786

Contributor

awd-studio commented Nov 4, 2017

Вот на YouTube классно реализована фича, давайте и у нас запилим?

Там при первом ПКМ открывается их контекстка, а при втором - нормальная.

Реализовал в #786

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Nov 4, 2017

Contributor

Теперь менюшка не закрывается если кликнуть в свободное пространство.

Contributor

mbaev commented Nov 4, 2017

Теперь менюшка не закрывается если кликнуть в свободное пространство.

@awd-studio

This comment has been minimized.

Show comment
Hide comment
@awd-studio

awd-studio Nov 4, 2017

Contributor

Аё...
Ща, поборю )))

Contributor

awd-studio commented Nov 4, 2017

Аё...
Ща, поборю )))

@mbaev

This comment has been minimized.

Show comment
Hide comment
@mbaev

mbaev Nov 4, 2017

Contributor

Работает отлично в chrome и mozilla!

Contributor

mbaev commented Nov 4, 2017

Работает отлично в chrome и mozilla!

@mbaev mbaev closed this Nov 4, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment