Несоответствия с двумя кнопками (меню поста)

awesomerobot · 03.Февраль.2025 14:31:49

This should fix both issues, and some others I noticed

github.com/discourse/discourse

UX: clean up post action menu styles, fix issues, add variables

main ← ux-post-action-menu

merged 02:31PM - 03 Feb 25 UTC

awesomerobot

+142 -127

Related to https://meta.discourse.org/t/double-button-inconsistencies-post-menu/…349845 This does some general clean up... * **Moves shared mobile/desktop styles into /common** We had some mobile hover states for some reason, and desktop hover/focus states can be moved to common and gated with `.discourse-no-touch`... this means we're applying them on capabilities rather than device type * **Adds some `-d-post-control-` variables to make theming easier** Theme authors can replace the variables without worrying about selector specificity * **Removes an overridden fade-out class from likes** We were overriding the effect here anyway * **Fixes a janky hover transition effect on the like button** This was being incorrectly inherited from another button ...and fixes some issues * **Corrects border radius on double buttons (likes, flags)** ![image](https://github.com/user-attachments/assets/e3bf45b4-3d72-4858-94b1-dad41ff3e274) * **Corrects double button height issues for flags** ![image](https://github.com/user-attachments/assets/eb5624ea-f434-43b6-8ffc-de9baf1fa0fd) * **Adjusts the flag count to avoid the lumpy circle CSS problem** ![image](https://github.com/user-attachments/assets/7c0d85bc-7d72-4b99-ab1a-ec45bab05750) * **Removes lingering post-tap focus/hover states on mobile by applying `.discourse-no-touch` and `focus-visible`** Tested both glimmer and legacy.

Тема		Ответов	Просм.
Updating Discourse Plugins, taking the first step Development	12	3805	22.03.2016
Some icons vanish when tabbed over Bug accessibility	2	620	26.05.2021
Double buttons in posting section? Puzzled Support	14	170	09.10.2024
Adjust the post-voting button to the bottom Theme component post-voting	10	989	21.01.2025
Post action menu is cut off on mobile UX	3	837	04.06.2021

Несоответствия с двумя кнопками (меню поста)

Связанные темы