Skip to content

Add tooltip to attribute type#1447

Draft
amomchilov wants to merge 1 commit into
ruby:masterfrom
amomchilov:attr-tooltip
Draft

Add tooltip to attribute type#1447
amomchilov wants to merge 1 commit into
ruby:masterfrom
amomchilov:attr-tooltip

Conversation

@amomchilov

@amomchilov amomchilov commented Nov 7, 2025

Copy link
Copy Markdown
  1. Move the [R] part outside the underlined and linked <a> tag

  2. Add a title field to explain what [R], [W] or [RW] mean.

    It's fairly self-explanatory if you see [RW], but on many APIs' docs, you'll only have a bunch of [R]s, so you'll be missing that context clue.

    On hover, it shows one of:

    • attr_reader
    • attr_writer
    • attr_accessor
Before After
Before After

@amomchilov amomchilov temporarily deployed to fork-preview-protection November 7, 2025 18:12 — with GitHub Actions Inactive
@matzbot

matzbot commented Nov 7, 2025

Copy link
Copy Markdown
Collaborator

🚀 Preview deployment available at: https://7eff1332.rdoc-6cd.pages.dev (commit: 1468aea)

Comment thread lib/rdoc/generator/template/darkfish/class.rhtml Outdated
@michaelherold

Copy link
Copy Markdown

The behavior of the is quite confusing with this change. It pops up when you hover over anything in the div, but appears to be associated with the a. I'm not really sure what its purpose is as there is no click/select behavior attached to it. My guess is that it's there to make hovering the link more obvious.

I like the change, but think we should address the pilcrow's (TIL its name) behavior as part of it.

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.

3 participants