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

Add delete option to Library mods kebab menu#3745

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
Al12rs merged 3 commits intomainfromfeat/3743-kebab-menu-installed-mods
Aug 20, 2025

Conversation

@Al12rs
Copy link
Contributor

@Al12rsAl12rs commentedAug 20, 2025•
edited
Loading

Added a dedicatedLibraryItemIdsCompositItemModel component that could be used to get the Ids of the current item, decoupling that from theInstallAction component.

QA:

  • Go to library page
  • On the right of library entries there should be a three dots menu button
  • Verify that delete button correctly removes items as expected
    • For child entries
    • For parent entries with multiple children

@Al12rsAl12rs requested a review froma teamAugust 20, 2025 09:05
@Al12rsAl12rs self-assigned thisAug 20, 2025
@Al12rsAl12rs added this toMVPAug 20, 2025
@Al12rsAl12rs moved this toReview inMVPAug 20, 2025
Copy link
Member

@Sewer56Sewer56 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

Just had a quick look/overview. Nothing too intense.
Looks reasonable to me.

@Al12rsAl12rs merged commit3894fd0 intomainAug 20, 2025
10 checks passed
@github-project-automationgithub-project-automationbot moved this fromReview toDone inMVPAug 20, 2025
@Al12rsAl12rs deleted the feat/3743-kebab-menu-installed-mods branchAugust 20, 2025 10:24
@Al12rsAl12rs added the Needs QAFor tickets that should go through the QA validation process labelAug 27, 2025
@Al12rsAl12rs moved this fromDone toReady for QA inMVPAug 27, 2025
@RedRanger14RedRanger14 moved this fromQA - Ready toQA - In progress inMVPAug 27, 2025
@RedRanger14RedRanger14 self-assigned thisAug 27, 2025
@RedRanger14RedRanger14 moved this fromQA - In progress toQA - Done inMVPAug 27, 2025
@RedRanger14
Copy link

Minor visual errors to be picked up by another ticket

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

Reviewers

@Sewer56Sewer56Sewer56 approved these changes

Labels

Needs QAFor tickets that should go through the QA validation process

Projects

Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Al12rs@RedRanger14@Sewer56
[8]ページ先頭
©2009-2025 Movatter.jp