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

Add ruby-title class to method name in HTML #555

Merged
merged 1 commit into from
Nov 21, 2017

Conversation

aycabta
Copy link
Member

@aycabta aycabta commented Nov 11, 2017

RDoc::RipperStateLex#get_op_tk set :on_ident to method name token before RDoc::TokenStream.to_html, so the RDoc::TokenStream.to_html can detect method name token.

This commit changes behavior of markup to add ruby-title class to method name in HTML.

This was the behavior of SDoc' monkey patch what is removed at https://github.com/zzak/sdoc/pull/112, and RDoc's RDoc::TokenStream.to_html can only implement it.

RDoc::RipperStateLex#get_op_tk set :on_ident to method name token before
RDoc::TokenStream.to_html, so RDoc::TokenStream.to_html can detect
method name token.

This commit changes behavior of markup to add ruby-title class to method
name in HTML.
@hsbt hsbt merged commit 1d94023 into ruby:master Nov 21, 2017
@aycabta aycabta deleted the set-ruby-title-to-method-name branch November 21, 2017 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants