Make link fields default to render as link via a slot

Created on 17 January 2023, over 1 year ago
Updated 26 January 2023, over 1 year ago

Currently link fields use the default TextFieldItemProcessor, which does not format links according to display settings.

Add a new processor for Link fields which does.

Patch to follow.

✨ Feature request
Status

Fixed

Version

3.0

Component

Code

Created by

πŸ‡¬πŸ‡§United Kingdom Leo Pitt

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.

  • Status changed to Needs work over 1 year ago
  • πŸ‡¦πŸ‡ΉAustria fago Vienna

    I agree this is a better default, but technically changing the default is BC break, since people might be using the current field-link.

    Generally, I think it's not ideal to use separate slot for simple field like that, but add the link-data as prop so the frontend can freely position and render the link. What would you think about that Leo?

    Setting to needs work since we need to discuss the BC issue.

  • Status changed to RTBC over 1 year ago
  • πŸ‡¦πŸ‡ΉAustria fago Vienna

    we discussed in the meeting to move those improvements into 3.x, so we don't have an issue with BC breaks.

  • Status changed to Fixed over 1 year ago
  • πŸ‡ΊπŸ‡¦Ukraine mostepaniukvm

    Merged into 3.x branch. Also synced 3.x with the latest changes in 8.x-2x

  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.69.0 2024