Skip to content

Feat/hide delete button#2961

Merged
alfdocimo merged 4 commits intomasterfrom
feat/hide-delete-button
Feb 11, 2026
Merged

Feat/hide delete button#2961
alfdocimo merged 4 commits intomasterfrom
feat/hide-delete-button

Conversation

@alfdocimo
Copy link
Contributor

Category/Component

Adding a new prop to hide the delete button in the molecule photo uploader 📸

TASK:

Description, Motivation and Context

Types of changes

  • 🪲 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🧾 Documentation
  • 📷 Demo
  • 🧪 Test
  • 🧠 Refactor
  • 💄 Styles
  • 🛠️ Tool

Screenshots - Animations

Screenshot 2026-02-09 at 16 17 40

@github-actions
Copy link

github-actions bot commented Feb 9, 2026

STATEMENTS BRANCHES FUNCTIONS LINES
≍ 0= ≍ 0.05↓ ≍ 0= ≍ 0=
% 76.54 63.54 68.15 78.08
ABS 3488 / 4557 2118 / 3333 685 / 1005 3307 / 4235

errorFormatPhotoUploadedText,
errorInitialPhotoDownloadErrorText,
errorSaveImageEndpoint,
hideDeleteButton = false,
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't need a default value here.

Copy link
Member

Choose a reason for hiding this comment

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

preferible in semantical affirmative

Suggested change
hideDeleteButton = false,
showDeleteButton = true,

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated 👍

@github-actions
Copy link

STATEMENTS BRANCHES FUNCTIONS LINES
≍ 0= ≍ 0.05↓ ≍ 0= ≍ 0=
% 76.54 63.54 68.15 78.08
ABS 3488 / 4557 2118 / 3333 685 / 1005 3307 / 4235

@alfdocimo alfdocimo merged commit 502ec85 into master Feb 11, 2026
7 checks passed
@alfdocimo alfdocimo deleted the feat/hide-delete-button branch February 11, 2026 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants