Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

refactor(site): move toggle favorites handler inside ToggleFavoriteButton #11892

Closed as not planned
Assignees
johnstcn
Labels
siteArea: frontend dashboardstaleThis issue is like stale bread.
@johnstcn

Description

@johnstcn
          It's an implicit convention, but when dealing with event handlers, we usually name them using the `on` prefix like `onSomething` instead of `handleSomething`. So instead of `handleActions`, I think a better name would be `onToggle`.

Also, you can move the query inside this button 😁. We're trying to put things close to where they're used.

Originally posted by@BrunoQuaresma in#11875 (comment)

Metadata

Metadata

Assignees

Labels

siteArea: frontend dashboardstaleThis issue is like stale bread.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions


    [8]ページ先頭

    ©2009-2025 Movatter.jp