Thank you for taking the time to contribute to this project! Please take the time to tell us a bit about the changes you've made.
Description
Give a short and brief description of the pull request. Add a screenshot if appropriate and helpful. Changes can be listed in the next section.
Program the "Share/save loop" button to display a modal with a link to share the loop. Additionally, add auto-select and auto-copy features for input fields.
Closes #6 and #84.
Previews
Changes
List the changes made.
Added a modal with a share link, displayed when clicking the "Share/save loop" button
Added custom data attributes added for auto-selecting and auto-copying text in input elements
Added custom tooltip component added
Added auto-select to video link/ID input and share link input fields
Added auto-copy to share link input field (requires SSL certificate for this to work on mobile browsers)
Added event handler for input focus event, to handle auto-select and auto-copy
Changed background color of readonly and disabled form elements
Changed jQuery's document ready event handler to use function () instead of arrow function, as it is recommended and gives access to $(this) should we need it in the future
Breaking Change?
Will these changes cause existing functionality to not work as expected? Will contributors be able to run the project after these changes are merged, without needing to take any additional steps?
[ ] Please tick if this is a breaking change and describe why below
Merging Checklist
Lastly, before merging we need to make sure that these are done.
Pull Request Submission
Thank you for taking the time to contribute to this project! Please take the time to tell us a bit about the changes you've made.
Description
Program the "Share/save loop" button to display a modal with a link to share the loop. Additionally, add auto-select and auto-copy features for input fields.
Closes #6 and #84.
Previews
Changes
function ()
instead of arrow function, as it is recommended and gives access to$(this)
should we need it in the futureBreaking Change?
Merging Checklist
All tests are up-to-dateN/AAll tests passN/A