Skip to content
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

Ligatures documentation: Screenshots too generic, not depicting the relevant intermediary step(s) of the transformation #20407

Open
1 of 2 tasks
porg opened this issue Nov 7, 2024 · 0 comments

Comments

@porg
Copy link

porg commented Nov 7, 2024

Bug description

Feedback on article "Using Ligatures"

https://docs.fontawesome.com/desktop/add-icons/ligatures

Status Quo

Section: Set the Font/Typeface to Font Awesome 6

https://docs.fontawesome.com/desktop/add-icons/ligatures#set-the-fonttypeface-to-font-awesome-6

  • Shows a screenshot with a text layer with the content: "Plain Old Text Layer"
    ligatures-2 CEHorkZN_1VVwbx

Section: Type the Icon’s Name

https://docs.fontawesome.com/desktop/add-icons/ligatures#type-the-icons-name

  • Shows a screenshot with a text layer with the rendered ligature, which is the rocket icon.

ligatures-3 C-XRsAgz_HlSOQ

Critique

  • ❌ The intermediary step is not depicted. Just the generic screenshot of a text layer.
  • ❌ You get no idea how this works in practice if you have yet not seen it in action.

Proposed improvement

  • 💡 Text layer with "rocke" and when the last letter in the ligature "t" is typed it transforms into the rocket icon.
    • As a intermediary screenshot.
    • Or better a short video clip typing the ligature and seeing how it gets transformed on the fly when typed completely.

Font Awesome version

v6

Application and Operating System

  • Feedback on your documentation

Web bug report checklist

  • I have included a test case because my odds go way up that the team can fix this when I do
  • I have searched for existing issues and to the best of my knowledge this is not a duplicate
@porg porg added bug needs-triage This bug needs to be confirmed labels Nov 7, 2024
@tagliala tagliala added doc enhancement and removed bug needs-triage This bug needs to be confirmed labels Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants