Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bookmark delete button text change #13842

Closed
HamagenCZE opened this issue Sep 21, 2023 · 8 comments
Closed

Bookmark delete button text change #13842

HamagenCZE opened this issue Sep 21, 2023 · 8 comments
Labels
pending closure Requires immediate attention to avoid being closed for inactivity type: feature Introduction of new functionality to the application

Comments

@HamagenCZE
Copy link

NetBox version

v3.6.2

Feature type

Change to existing functionality

Proposed functionality

I suggest changing the notification window when the Delete button is pressed in /user/bookmarks/ for the object.

At the moment the text is "Are you sure you want to delete #OBJECT NAME#?: Which evokes the feeling that the entire object will be deleted and not the bookmark itself.

I suggest changing the text to ""Are you sure you want to delete your bookmark for #OBJECT NAME#?"

Use case

My colleague and I, who manage Netbox, both had the same feeling that the button would delete the entire object instead of the bookmark. We were forced to make a dummy object, bookmark it and then test the button to be sure. This change, I think, will help the clarity of the button.

Database changes

No response

External dependencies

No response

@HamagenCZE HamagenCZE added the type: feature Introduction of new functionality to the application label Sep 21, 2023
@ITJamie
Copy link
Contributor

ITJamie commented Sep 21, 2023

Related to #13690

@abhi1693
Copy link
Member

@ITJamie could you please explain how it's related? I don't see any relation. This FR is simply asking for a delete message update whereas yours is changing how the delete works by showing related objects.

@ITJamie
Copy link
Contributor

ITJamie commented Sep 22, 2023

See the screenshots i added in that fr. It shows exactly what would be deleted including the primay object itself.

If you were deleting just a bookmark it would say "bookmark - bookmark name" in the list of items that would be deleted

@HamagenCZE
Copy link
Author

I dont see a relation too. You are talking about normal delate behavior and about object deletion and relative links to that object, my propose was only for changing of text to clarify function of a bin icon in Bookmarks list which is not a object deletion at all

@ITJamie
Copy link
Contributor

ITJamie commented Sep 29, 2023

Sorry for the delay I was on vacation.
The initial version of a branch I made for listing objects that would be deleted included the primary item.

My view was that the delete modal right now does not give enough details and that this issue is related heavily to the other issue.

I do think it's odd that the {{ object_type }} isn't rendering for bookmark objects though...

here is a screenshot of what the output of the delete form would look like with my current dev branch
Screenshot 2023-09-29 at 21 21 02

The text etc could be re-worked but I'm of the view that the delete modal needs to present as much information as possible

@ITJamie
Copy link
Contributor

ITJamie commented Sep 29, 2023

I found the reason as to why {{ object_type }} doesn't show on the delete page for bookmarks.
The following commit would solve it. 3d9a252

the htmx render call has the object_type variable set but the non-htmx render call does not

Copy link
Contributor

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. NetBox is governed by a small group of core maintainers which means not all opened issues may receive direct feedback. Do not attempt to circumvent this process by "bumping" the issue; doing so will result in its immediate closure and you may be barred from participating in any future discussions. Please see our contributing guide.

@github-actions github-actions bot added the pending closure Requires immediate attention to avoid being closed for inactivity label Dec 29, 2023
@ITJamie ITJamie mentioned this issue Jan 20, 2024
32 tasks
Copy link
Contributor

This issue has been automatically closed due to lack of activity. In an effort to reduce noise, please do not comment any further. Note that the core maintainers may elect to reopen this issue at a later date if deemed necessary.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jan 28, 2024
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
pending closure Requires immediate attention to avoid being closed for inactivity type: feature Introduction of new functionality to the application
Projects
None yet
Development

No branches or pull requests

3 participants