rouge icon indicating copy to clipboard operation
rouge copied to clipboard

Make prefix of CSS classes for tokens configurable

Open mojavelinux opened this issue 6 years ago • 6 comments

Currently, the token shortname is used directly as the CSS class for a token. Due to the brevity of the shortnames (e.g., err), when the Rouge stylesheet is combined with other stylesheets in a site, this can lead to styling conflicts.

I'd like to propose that the CSS classes prefix for tokens be made configurable. I suggest using the name token_class_prefix (e.g., token_class_prefix: 'tok-')

It's relevant to note that Pygments offers this option (as classprefix), so this could prove important for Pygments compatibility as well.

mojavelinux avatar Jan 23 '19 09:01 mojavelinux

This issue has been automatically marked as stale because it has not had any activity for more than a year. It will be closed if no additional activity occurs within the next 14 days. If you would like this issue to remain open, please reply and let us know if the issue is still reproducible.

stale[bot] avatar May 30 '20 02:05 stale[bot]

I'm happy to send a PR for this issue, but I'm waiting on a decision about the name of the option. Is my proposal acceptable?

mojavelinux avatar May 31 '20 10:05 mojavelinux

@jneen What do you think of the tok- prefix idea? Seems like a good idea to me.

pyrmont avatar Jun 02 '20 18:06 pyrmont

This issue has been automatically marked as stale because it has not had any activity for more than a year. It will be closed if no additional activity occurs within the next 14 days. If you would like this issue to remain open, please reply and let us know if the issue is still reproducible.

stale[bot] avatar Jun 03 '21 20:06 stale[bot]

I'm still willing to send a PR if I can get confirmation this is an acceptable proposal.

mojavelinux avatar Jun 03 '21 20:06 mojavelinux

This issue has been automatically marked as stale because it has not had any activity for more than a year. It will be closed if no additional activity occurs within the next 14 days. If you would like this issue to remain open, please reply and let us know if the issue is still reproducible.

stale[bot] avatar Jun 12 '22 20:06 stale[bot]