Revert "Fix/Retention help button bounds (#4253)"

This reverts commit 5462d99255.
This commit is contained in:
Damien Elmes 2025-08-09 16:45:07 +10:00 committed by GitHub
parent a0c1a398f4
commit 1bcbf77ebb
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 20 additions and 16 deletions

View file

@ -12,7 +12,7 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
export { className as class }; export { className as class };
export let title: string; export let title: string;
export let onHelpClick: ((_e: MouseEvent | KeyboardEvent) => void) | null = null; export let onTitleClick: ((_e: MouseEvent | KeyboardEvent) => void) | null = null;
</script> </script>
<div <div
@ -25,21 +25,25 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
style:--container-margin="0" style:--container-margin="0"
> >
<div class="position-relative"> <div class="position-relative">
<h1> {#if onTitleClick}
{title} <span
</h1> on:click={onTitleClick}
{#if onHelpClick} on:keydown={onTitleClick}
<div
on:click={onHelpClick}
on:keydown={onHelpClick}
role="button" role="button"
tabindex="0" tabindex="0"
class="help-badge position-absolute"
class:rtl
> >
<slot name="tooltip" /> <h1>
</div> {title}
</h1>
</span>
{:else}
<h1>
{title}
</h1>
{/if} {/if}
<div class="help-badge position-absolute" class:rtl>
<slot name="tooltip" />
</div>
</div> </div>
<slot /> <slot />
</div> </div>

View file

@ -8,7 +8,7 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
// When title is null (default), the graph is inlined, not having TitledContainer wrapper. // When title is null (default), the graph is inlined, not having TitledContainer wrapper.
export let title: string | null = null; export let title: string | null = null;
export let subtitle: string | null = null; export let subtitle: string | null = null;
export let onHelpClick: ((_e: MouseEvent | KeyboardEvent) => void) | null = null; export let onTitleClick: ((_e: MouseEvent | KeyboardEvent) => void) | null = null;
</script> </script>
{#if title == null} {#if title == null}
@ -19,7 +19,7 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
<slot /> <slot />
</div> </div>
{:else} {:else}
<TitledContainer class="d-flex flex-column" {title} {onHelpClick}> <TitledContainer class="d-flex flex-column" {title} {onTitleClick}>
<slot slot="tooltip" name="tooltip"></slot> <slot slot="tooltip" name="tooltip"></slot>
<div class="graph d-flex flex-grow-1 flex-column justify-content-center"> <div class="graph d-flex flex-grow-1 flex-column justify-content-center">
{#if subtitle} {#if subtitle}

View file

@ -57,12 +57,12 @@ License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
const title = tr.statisticsTrueRetentionTitle(); const title = tr.statisticsTrueRetentionTitle();
const subtitle = tr.statisticsTrueRetentionSubtitle(); const subtitle = tr.statisticsTrueRetentionSubtitle();
const onHelpClick = () => { const onTitleClick = () => {
openHelpModal(Object.keys(retentionHelp).indexOf("trueRetention")); openHelpModal(Object.keys(retentionHelp).indexOf("trueRetention"));
}; };
</script> </script>
<Graph {title} {subtitle} {onHelpClick}> <Graph {title} {subtitle} {onTitleClick}>
<HelpModal <HelpModal
title={tr.statisticsTrueRetentionTitle()} title={tr.statisticsTrueRetentionTitle()}
url={HelpPage.DeckOptions.fsrs} url={HelpPage.DeckOptions.fsrs}