add properties for back/forward mouse buttons to button and eventbox

[end-4]

For mice with side buttons

• Adds the following props to Widget.Button and Widget.EventBox

  • on-back-click
  • on-forward-click
  • on-back-click-release
  • on-forward-click-release

• Note: There doesn't seem to be Gdk consts for values 8 and 9 so I declared consts for them. Hopefully that's alright

[Aylur]

I am kind of hesitant to add this, this makes sense based on the pattern I followed, but maybe we should make on_clicked the only property and make it more generic?

[end-4]

I guess. Someone in discord told me about mice with 12 side buttons lol
Though no one likes breaking changes...

[NicolaiVdS]

What imo would make sense is adding a on-middle-click and on-middle-click-release

[end-4]

What imo would make sense is adding a on-middle-click and on-middle-click-release

what
https://aylur.github.io/ags-docs/config/widgets/#button

[NicolaiVdS]

What imo would make sense is adding a on-middle-click and on-middle-click-release

what https://aylur.github.io/ags-docs/config/widgets/#button

ok nevermind looked over it

[end-4]

maybe we should make on_clicked the only property and make it more generic?

• I realized on_clicked is already a thing currently, so I suggest the name on_mouse_button
• A more generic prop would probably make ags' code cleaner, but would that make it less convenient for users? I'm not sure since I don't know a lot of Js.

[Aylur]

the most convenient is probably adding this as it is
I'm thinking about something like

type State = "clicked" | "released"
type Button = "primary" | "secondary" | "middle" | "back" | "forward"

Button({
  onMouseButton(self, opts: { event: Gdk.Event, state: State, button: Button }) {
    if (state === "clicked" && button === "primary") {
      // onPrimaryClickRelease
    }
  }
})

but the issue with this is that it would require bloat to check for the clicked state, when usually only the clicked state is desired
so maybe on-mouse-button and on-mouse-button-released?