Skip to content

Media browser updates#6801

Merged
zsarnett merged 4 commits intodevfrom
media-browser-updates
Sep 6, 2020
Merged

Media browser updates#6801
zsarnett merged 4 commits intodevfrom
media-browser-updates

Conversation

@zsarnett
Copy link
Copy Markdown
Contributor

@zsarnett zsarnett commented Sep 6, 2020

Proposed change

Add sorting to sort folders first and then by title

Creates an error box if we receive an error when browsing (ex. user tries to browse a file that doesn't exist. This can be achieved if the user browses using the media browser and then deletes the file and tries to browse to it)

fixes the position of the dialog on the screen so that the top of the dialog stays at the same spot and only the height changes

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (thank you!)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Example configuration

Additional information

  • This PR fixes or closes issue: fixes #
  • This PR is related to issue:
  • Link to documentation pull request:

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

Comment on lines +416 to +420
: first.title > second.title
? 1
: second.title > first.title
? -1
: 0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@zsarnett zsarnett merged commit efe8eca into dev Sep 6, 2020
@zsarnett zsarnett deleted the media-browser-updates branch September 6, 2020 23:28
@bramkragten bramkragten mentioned this pull request Sep 7, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Jul 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants