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

Fragment Copy Button conflicts with Doxygen copy button #142

Closed
CaseyNelson314 opened this issue May 19, 2024 · 2 comments · Fixed by #143
Closed

Fragment Copy Button conflicts with Doxygen copy button #142

CaseyNelson314 opened this issue May 19, 2024 · 2 comments · Fixed by #143

Comments

@CaseyNelson314
Copy link
Contributor

Hi, thanks for the great project!

I found a small problem and would like to report it.

Problem

Using the extension "Fragment Copy Button" conflicts with Doxygen copy button as shown in the screenshot.

screenshot

You can see the conflict from this page. https://github.com/CaseyNelson314/doxygen-page-demo

The version I am using is Doxygen 1.10.0.

Solution

Set HTML_COPY_CLIPBOARD to NO in Doxyfile. The default is YES.

This option was added in 1.10.0. https://www.doxygen.nl/manual/changelog.html#log_1_10_0

Suggestions

I think we should add a note to extensions.md that HTML_COPY_CLIPBOARD should be NO when using Fragment Copy Button.

Other

This project page is also using Doxygen 1.10.0, yet I am not experiencing any problems. This is probably because the project uses the old Doxygen-generated header.html, which does not contain any clipboard-related code.

In Doxygen 1.10.0, the following code is added when generating header.html

<script type="text/javascript" src="$relpath^clipboard.js"></script>

This is a translated text. Some parts may be difficult to understand. We apologize in advance.

@kitswas
Copy link

kitswas commented Oct 7, 2024

I was wondering if, with the Doxygen copy button, the "Fragment Copy Button" extension still needs to exists.
If not, deprecating the extension might reduce @jothepro's maintenance load.

See https://github.com/doxygen/doxygen/blob/1822879cdf4a7a08bd75870ea9257bd2946e56ca/templates/html/clipboard.js#L3-L4

@jothepro
Copy link
Owner

That's a fair question! I will evaluate if the code can be removed.

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 a pull request may close this issue.

3 participants