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

Adding shortcuts dashboard widget #14571

Open
wants to merge 30 commits into
base: 3.x
from

Conversation

Projects
None yet
7 participants
@sdrenth
Copy link
Contributor

commented Apr 25, 2019

What does it do?

Adds a shortcuts widget to the dashboard with an overview of all MODX related shortcuts.

Why is it needed?

Provides a handy overview for every MODX user to speed up their workflow with MODX.

Note: I've added the newObject in dashboard_widgets.php with array key 9 instead of 8 so it can be easily merged together with PR: #14569

Screenshot 2019-04-25 at 14 41 14

ilyautkin and others added some commits Apr 10, 2019

Merge pull request #4 from sdrenth/fix-improving-mediabrowser-handling
If a file already exists generate a unique filename and upload as new…
Merge pull request #3 from joeke/3.x
Fix width on modx-leftbar, give more room to resources/elements/files…
Merge pull request #6 from wax100/fix-mediasource-downloading
Fix Downloading from Media Source
Merge pull request #9 from joeke/media-browser-reload
Add extra check for browser variable

@sdrenth sdrenth requested review from Mark-H and opengeek as code owners Apr 25, 2019

@gpsietzema

This comment has been minimized.

Copy link
Collaborator

commented Apr 25, 2019

❤️ on this post if you also didn't know about the cache clearing shortcut lol :-D

@Ruslan-Aleev

This comment has been minimized.

Copy link
Contributor

commented Apr 25, 2019

Oooh, this is a really useful UX thing, thanks!
And this gives a reminder that new shortcuts can be created in MODX.

gpsietzema and others added some commits Apr 25, 2019

Revert "Merge branch '3.x' into feature-shortcuts-widget"
This reverts commit 32fe659, reversing
changes made to 76d8f22.
@JoshuaLuckers

This comment has been minimized.

Copy link
Collaborator

commented Apr 25, 2019

This widget is very useful! Are there more shortcuts that are not yet listed here available?

@Ruslan-Aleev

This comment has been minimized.

Copy link
Contributor

commented Apr 25, 2019

However, need to consider that the combinations in Mac OS and Windows are different, maybe should write this:

Ctrl / Cmd + Shift + U
...

@JoshuaLuckers

This comment has been minimized.

Copy link
Collaborator

commented Apr 25, 2019

That’s already the case @Ruslan-Aleev 👍

@Ruslan-Aleev

This comment has been minimized.

Copy link
Contributor

commented Apr 25, 2019

Yes, I apologize, I saw in the code, at first I did not pay attention :)

@sdrenth

This comment has been minimized.

Copy link
Contributor Author

commented Jun 14, 2019

@Mark-H @opengeek Can you please review this PR so we can get this PR merged?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.