Skip to content

Missing selector cursor icons in example apps #27

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

Open
dackers86 opened this issue Apr 17, 2025 · 1 comment
Open

Missing selector cursor icons in example apps #27

dackers86 opened this issue Apr 17, 2025 · 1 comment
Labels
bug Something isn't working

Comments

@dackers86
Copy link
Member

Hovering over the following actions, does not display a cursor-pointer icon.

screens/sign-in-auth-screen - NextJS & Angular forgotten password.
screens/sign-in-auth-screen - NextJS terms of service links.
screens/sign-in-auth-screen - NextJS sign-out button

@dackers86
Copy link
Member Author

dackers86 commented Apr 17, 2025

Add the cursor icon to the following could be a component wide solution:

 .fui-form .fui-form__action {
    @apply px-1 hover:underline text-xs text-text-muted cursor-pointer;
  }

The policies component in NextJS would benefit from:

<div className="text-text-muted text-xs text-start">
      {parts.map((part: string, index: number) => {
        if (part === "{tos}") {
          return (
            <a
              key={index}
              onClick={() => handleUrl(termsOfServiceUrl)}
              target="_blank"
              rel="noopener noreferrer"
              className="text-text-muted hover:underline font-semibold" <--- Add cursor-pointer here
            >
              {termsText}
            </a>
          );
        }
        if (part === "{privacy}") {
          return (
            <a
              key={index}
              onClick={() => handleUrl(privacyPolicyUrl)}
              target="_blank"
              rel="noopener noreferrer"
              className="text-text-muted hover:underline font-semibold"  <--- Add cursor-pointer here
            >
              {privacyText}
            </a>
          );
        }
        return <span key={index}>{part}</span>;
      })}
    </div>

The NextJS Sign-uut button could benefit with something similar to Angulars header-component:

    button {
      background: none;
      border: none;
      cursor: pointer;  <-- This line
      font: inherit;
      color: inherit;
    }

@dackers86 dackers86 added the bug Something isn't working label Apr 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant