Change Copy link button selector

Created on 2 September 2022, almost 2 years ago
Updated 1 June 2023, about 1 year ago

Problem/Motivation

copy-current-url.js uses 'a.btnCopy' selector, but I have instead of link on the page.

Proposed resolution

We should change selector in copy-current-url.js file.

๐Ÿ› Bug report
Status

Closed: works as designed

Version

4.0

Component

Code

Created by

๐Ÿ‡ฆ๐Ÿ‡ฒArmenia levmyshkin Yerevan, Armenia

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • ๐Ÿ‡ท๐Ÿ‡บRussia Gal K

    btnCopy is not an <a> tag. It's a <button> html tag. levmyshkin โ†’ , thanks for the patch

  • ๐Ÿ‡บ๐Ÿ‡ธUnited States shelane

    Actually, itโ€™s not and the change was made back in 2021 before this issue was made. Please check your version and see that you arenโ€™t using a custom template in your theme.

  • ๐Ÿ‡ท๐Ÿ‡บRussia Gal K

    Please check your version and see that you arenโ€™t using a custom template in your theme.

    Yes, you are right, it's because of my custom template. My bad

  • Status changed to Closed: works as designed about 1 year ago
  • ๐Ÿ‡บ๐Ÿ‡ธUnited States shelane

    It looks like levmyshkin is as well. Please update your custom templates as other changes were made as well.

Production build 0.69.0 2024