-
Notifications
You must be signed in to change notification settings - Fork 1
Add Modal for Modifying or Deleting Existing Gold Entries #39
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
base: master
Are you sure you want to change the base?
Conversation
| } | ||
|
|
||
|
|
||
| function make_gold_list(golds) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please also delete the old version of make_gold_list
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mr-martian the changes are made.
static/regtest.js
Outdated
|
|
||
| $(window).on('resize scroll', event_scroll); | ||
| }); | ||
| }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All lines must be terminated. Do not remove the trailing newlines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@TinoDidriksen trailing newline is added at the end of the file.
Overview
This PR introduces functionality to edit and delete gold entries within the corpus. Users can now interact with individual gold entries and either modify or remove them as needed.
Changes Made:
btn_gold_edit():btn_gold_delete():Event Handling in
cb_load():.rt-gold lielements.btn_gold_editorbtn_gold_delete) based on the user's choice.Implementation Details:
Editing Gold:
Deleting Gold:
Why This Change Is Important:
Related Issue: