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

Allow section headings to be customized #316

Merged
merged 4 commits into from
Jul 18, 2021
Merged

Allow section headings to be customized #316

merged 4 commits into from
Jul 18, 2021

Conversation

BSaifullah
Copy link
Contributor

Closes #305

Section headings can now be given custom names through `dashboard-item-names'.
Since this is done through overlays it doesn't break any function that expects
the headings to have their default names.

It would arguably be more elegant to adapt all of the aforementioned cases to
use a system that can handle any name, but the work necessary would be a lot
more extensive than this, and I don't know how much of the codebase one would
want to revamp to accommodate this single feature.

Section headings can now be given custom names through `dashboard-item-names'.
Since this is done through overlays it doesn't break any function that expects
the headings to have their default names.
Copy link
Member

@jcs090218 jcs090218 left a comment

Choose a reason for hiding this comment

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

Can you make these changes for us? Thanks!

dashboard-widgets.el Outdated Show resolved Hide resolved
Also fix the issue with `dashboard-center-content' by using the correct
marker insertion type when making the overlay.
@jcs090218
Copy link
Member

Can you fix the CI for us?

And I forgot to mentioned, can you add document to our README file so the user would know this nice new feature? 😁 Thanks!

@BSaifullah
Copy link
Contributor Author

Done :-)

Copy link
Member

@jcs090218 jcs090218 left a comment

Choose a reason for hiding this comment

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

Thank you for contribute to emacs-dashboard. 😁

@jcs090218 jcs090218 merged commit 88c9ac0 into emacs-dashboard:master Jul 18, 2021
@BSaifullah BSaifullah deleted the feature/custom-headings branch September 13, 2021 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Customizable headlines in the widget
2 participants