Skip to content

Commit

Permalink
fix with-imported-switch for safari (plausible#4194)
Browse files Browse the repository at this point in the history
  • Loading branch information
RobertJoonas authored Jun 6, 2024
1 parent 2a529b4 commit b49bbbc
Showing 1 changed file with 12 additions and 6 deletions.
18 changes: 12 additions & 6 deletions assets/js/dashboard/stats/graph/with-imported-switch.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,24 +4,30 @@ import * as url from '../../util/url'
import { BarsArrowUpIcon } from '@heroicons/react/20/solid'
import classNames from "classnames"

function LinkOrDiv({isLink, target, children}) {
if (isLink) {
return <Link to={target}>{ children }</Link>
} else {
return <div>{ children }</div>
}
}

export default function WithImportedSwitch({query, info}) {
if (info && info.visible) {
const {togglable, tooltip_msg} = info
const enabled = togglable && query.with_imported
const target = url.setQuery('with_imported', (!enabled).toString())

const linkClass = classNames({
const iconClass = classNames("mt-0.5", {
"dark:text-gray-300 text-gray-700": enabled,
"dark:text-gray-500 text-gray-400": !enabled,
"cursor-pointer": togglable,
"pointer-events-none": !togglable,
})

return (
<div tooltip={tooltip_msg} className="w-4 h-4 mx-2">
<Link to={target} className={linkClass}>
<BarsArrowUpIcon className="mt-0.5"/>
</Link>
<LinkOrDiv isLink={togglable} target={target}>
<BarsArrowUpIcon className={iconClass}/>
</LinkOrDiv>
</div>
)
} else {
Expand Down

0 comments on commit b49bbbc

Please sign in to comment.