Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Make the icons clickable with the file names and performance improvements #129

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

Merged
merged 3 commits into from
May 23, 2019
Merged

Conversation

seagle0128
Copy link
Contributor

  1. Fix Can not select item when using icons #128: Can not select item when using icons.
  2. Refactor: remove the advice for dashboard-insert-heading.
  3. Use lexical binding to improve performance.
  4. Eliminate compiler warnings.

Copy link
Contributor

@ema2159 ema2159 left a comment

Choose a reason for hiding this comment

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

Some changes you removed from line 338 remove some relevant features.

  • The bullets for the org agenda

  • The tramp feature, which prevents dashboard from crashing when the remote is not available and stablishng unnecesary connections just to check the remote files when it is available.

Also the when-let macro leads to linting errors in the emacs 25 CI tests

@seagle0128
Copy link
Contributor Author

My bad! I didn't realize it supports 25. Commit again.

@JesusMtnez JesusMtnez self-requested a review May 22, 2019 19:35
@JesusMtnez
Copy link
Contributor

Thank you @seagle0128 and @ema2159 for all your work! 👍

@ema2159
Copy link
Contributor

ema2159 commented May 22, 2019

Thank you @seagle0128 and @ema2159 for all your work! 👍

This is a pretty nice and useful package, its fun to work with it!

Fix: CI issue on Emacs 25.
Fix: Unused lexical argument `list-size'.
Fix: deadlocks while visiting remote directories.
@ema2159
Copy link
Contributor

ema2159 commented May 23, 2019

Mr. @JesusMtnez, everything seems to be alright! Do your thing jaja you can merge this now

@JesusMtnez JesusMtnez merged commit 7a164ad into emacs-dashboard:master May 23, 2019
@seagle0128
Copy link
Contributor Author

Nice! Thank you, @JesusMtnez, @ema2159 and others!

@seagle0128 seagle0128 deleted the fix_128 branch May 23, 2019 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can not select item when using icons
3 participants