This repository has been archived by the owner on Dec 25, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 52
Adopt to new modern dialog #322
Merged
Merged
Changes from 53 commits
Commits
Show all changes
68 commits
Select commit
Hold shift + click to select a range
c847b77
Localization
nagmat84 c08de48
Changed dependency for basicModal
nagmat84 d894be1
Rewrote CSS
nagmat84 bcefc0c
Adopted jsDoc for basic modal
nagmat84 c9c497c
Adopted create login dialog
nagmat84 c44ad31
Adopted create (regular) album dialog
nagmat84 77d93e2
Adopted create (tag) album dialog
nagmat84 dcaab06
Adopted show tags dialog
nagmat84 b22e319
Adopted rename album dialog
nagmat84 5eaaac9
Adopted set album description dialog
nagmat84 61209cb
Adopted set album license dialog
nagmat84 2d47af6
Adopted set album soring dialog
nagmat84 6aa6fd7
Fix `basicModal.visible`
nagmat84 c5f5ed5
Adopted login and about dialog
nagmat84 19a5430
Renamed some methods
nagmat84 d7e67a7
Adopt Album Protection Policy Dialog
nagmat84 dd02e89
Make formatting
nagmat84 4fe43ee
Adopted Album Sharing Dialog
nagmat84 c704443
Adopted QR dialog
nagmat84 719d2af
Adopted Delete Album Dialog
nagmat84 7e01b80
Adopted Album Move/Merge Dialog
nagmat84 bb98a9b
Adopted Enter Password Dialog
nagmat84 63806d4
Adopt Import From URL Dialog
nagmat84 0c96733
Adopt Import From Server Dialog
nagmat84 ef3eddc
Adopt Upload Dialogs
nagmat84 51ddda8
Fix closing of dialogs
nagmat84 9252e8e
Adopted U2F dialog
nagmat84 36b0524
Adopted all dialogs but those for photos
nagmat84 9e4e49e
Adopted Photo Title Dialog
nagmat84 dba2c24
Adopted Photo Protection Policy Dialog
nagmat84 7538bc5
Adopted Photo Description Dialog
nagmat84 d2a2253
Adopted Photo Tag Dialog
nagmat84 26a2086
Adopted change from dialog to content area
nagmat84 d22b631
Adopted Photo Link Dialog
nagmat84 c90d014
Adopted Photo QR Code Dialog
nagmat84 29fce28
Adopted Photo Archive Dialog
nagmat84 1092b8d
Some cleanup
nagmat84 09ce761
Fix last dialog
nagmat84 1b7b6ec
Merge branch 'master' into adopt-new-modern-dialog
nagmat84 b0d3464
Fixed API token dialog
nagmat84 e6b33b8
Run prettier
nagmat84 5779f6d
Update scripts/3rd-party/basicModal.js
nagmat84 70a00b8
Apply suggestions from code review
nagmat84 f301e2d
Fix Typo "Model" -> "Modal"
nagmat84 66619a4
Fixed localization of sorting dialog
nagmat84 8279248
Remove unnecessary else-branch in initLogin
nagmat84 7065ddb
Fixed localization
nagmat84 b60fded
Fix error dialog for failed login
nagmat84 7ffd055
Apply suggestions from code review
nagmat84 7be3bea
Apply suggestion from code review.
nagmat84 d4bdecd
Removed unnecessaey !!
nagmat84 7391934
Merge branch 'master' into adopt-new-modern-dialog
nagmat84 692dbac
Removed jQuery leftover
nagmat84 a6a7cad
Apply suggestions from code review
nagmat84 4320beb
Made difference between layouts compact and compact-inverse more expl…
nagmat84 a9f5ba1
Apply suggestions from code review
nagmat84 f181f2c
Repair import from server (again)
nagmat84 2b83d2a
Merge branch 'master' into adopt-new-modern-dialog
nagmat84 9e941bb
Fixed typo
nagmat84 88fa3e6
Switched NPM dep for basicModal to v4.0.0
nagmat84 1211769
Switch to PR commit for basicModal to solve focussing problem
nagmat84 3b75276
Fix round corners of search box
nagmat84 e72d7a5
Repair button border in download dialog
nagmat84 30fe8df
Fixed mising space
nagmat84 79a9761
Always keep the last updated element of the upload list at the end
nagmat84 70701ef
Merge branch 'master' into adopt-new-modern-dialog
nagmat84 9a085f5
Tweak the borders and shadows in dialog boxes
kamil4 069b906
Fix localization of general error during upload
nagmat84 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I'm trying to decide whether to like this or to hate this 😜. It avoids us having to make "releases" of
basicModal
(like we currently have to do for php-exif on the server side), which seems like a big plus. Still, it's hard to be in favor of hardcoding a git commit... I don't know...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.
This isn't meant to be a permanent situation. This PR relies on some bug fixes to basicModal which are not yet merged into the
master
branch of basicModal but are part of the open PR LycheeOrg/basicModal#5. Hence, I needed to point to the tip of LycheeOrg/basicModal#5 during development of this PR as long as the upstream PR hasn't merged either.The idea is as follows: Squash and merge LycheeOrg/basicModal#5 into
master
of modalDialog first, make a release of modalDialog, then change the package file here and switch over to the new release of basicModal.Basically the situation here is comparable to the situation between Lychee-front and Lychee.
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.
Fixed in 88fa3e6. Now it is
after LycheeOrg/basicModal#8 has been merged and after I have created a new version tag after more than 4.5 years. 😎