Copy to Clipboard copies with the title

[harik8]

This is a Bug Report

When I click on copy to clipboard it copies with the title

pods/resource/cpu-request-limit.yaml

apiVersion: v1
kind: Pod
metadata:
  name: cpu-demo
  namespace: cpu-example
spec:
  containers:
  - name: cpu-demo-ctr
    image: vish/stress
    resources:
      limits:
        cpu: "1"
      requests:
        cpu: "0.5"
    args:
    - -cpus
    - "2"

Proposed Solution:
It'd be nice if it's only copies the yaml content.

Page to Update:
https://kubernetes.io/...

[sftim]

Thanks for reporting this @barik8
/kind bug
/priority important-longterm

[prasadkatti]

The code that copies the text to the clipboard is here -

website/static/js/script.js

Lines 20 to 52 in b3c961d

	function copyCode(elem){
	if (document.getElementById(elem)) {
	// create hidden text element, if it doesn't already exist
	var targetId = "_hiddenCopyText_";
	// must use a temporary form element for the selection and copy
	target = document.getElementById(targetId);
	if (!target) {
	var target = document.createElement("textarea");
	target.style.position = "absolute";
	target.style.left = "-9999px";
	target.style.top = "0";
	target.id = targetId;
	document.body.appendChild(target);
	}
	target.value = document.getElementById(elem).innerText;
	// select the content
	target.select();
	// copy the selection
	var succeed;
	try {
	succeed = document.execCommand("copy");
	} catch(e) {
	swal("Oh, no...","Sorry, your browser doesn't support document.execCommand('copy'), so we can't copy this code to your clipboard.");
	succeed = false;
	}
	if (succeed) swal("Copied to clipboard: ",elem);
	return succeed;
	} else {
	swal("Oops!",elem + " not found when trying to copy code");
	return false;
	}
	}

[prasadkatti]

This line -

website/static/js/script.js

Line 34 in b3c961d

target.value = document.getElementById(elem).innerText;

needs to be something like -
target.value = document.getElementById(elem).getElementsByTagName("tbody")[0].innerText;
in order to avoid the file name from the <thead> element. The elem here is the <table> element.

[tengqm]

@prasadkatti would you like to self assign this with a fix?

[fejta-bot]

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta.
/lifecycle stale

[kbhawkey]

A slightly different approach was taken to write the contents of the code block/element innerText.
I plan to close this issue. You can reopen if there are remaining questions.
Fixed in PR #23966 .
/close

[k8s-ci-robot]

@kbhawkey: Closing this issue.

Details

In response to this:

A slightly different approach was taken to write the contents of the code block/element innerText.
I plan to close this issue. You can reopen if there are remaining questions.
Fixed in PR #23966 .
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.