Small issue: the mouse out area from a button is smaller than the mouse in area

Canapin · September 7, 2022, 12:03pm

Replacing “mouseenter” by “mouseover” here:

app/assets/javascripts/discourse/app/components/scrolling-post-stream.js

c94a011d9


      
            };
          },
          
          willDestroyElement() {
            this._super(...arguments);
          
            document.removeEventListener("touchmove", this._debouncedScroll);
            window.removeEventListener("scroll", this._debouncedScroll);
            this.appEvents.off("post-stream:refresh", this, "_debouncedScroll");
            this.element.removeEventListener(
              "mouseenter",
              this._handleWidgetButtonHoverState
            );
            this.element.removeEventListener(
              "mouseleave",
              this._removeWidgetButtonHoverState
            );
            this.appEvents.off("post-stream:refresh", this, "_refresh");
            this.appEvents.off("post-stream:posted", this, "_posted");
          },

Resolves the issue, but I’m not sure if it has unwanted consequences or not.
I didn’t encounter any problem at first glimpse.

I suppose I’ll let the devs see if it’s a proper fix ot nor.

Topic		Replies	Views
Bug with the cursor and image Bug	4	1543	March 16, 2015
Wrong text color when hovering buttons on the 404 page UX	3	205	July 23, 2025
Double-button inconsistencies (post menu) UX	2	185	February 3, 2025
Search dropdown has layout shift on anon view UX	3	351	July 11, 2023
Reply button getting brightening and bold effect on scroll/hover Development	1	182	March 1, 2026

Small issue: the mouse out area from a button is smaller than the mouse in area

Related topics