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

feat: linkify note dialog note content #4886

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

Schwehn42
Copy link
Member

Description

Resolves #4885

Makes URLs visible and clickable in stack view, as is already the case for board view.
URLs should be editable and clickable after saving the changes again.

Change log

some refactoring was required to get this to work

  • Note Content: replace TextArea with contentEdtiable div
  • conditional content (linkified or pure)
  • URL style mixin
  • emoji suggestions adjustments (wip)

TODOs

  • cursor location when clicking div, changing to edit mode
  • emoji autocomplete (probably need to mock change event)

Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • The light- and dark-theme are both supported and tested
  • The design was implemented and is responsive for all devices and screen sizes
  • The application was tested in the most commonly used browsers (e.g. Chrome, Firefox, Safari)

(Optional) Visual Changes

Copy link

The deployment to the dev cluster was successful. You can find the deployment here: https://4886.development.scrumlr.fra.ics.inovex.io
This deployment is only for testing purposes and will be deleted after 1 week.
To redeploy rerun the workflow.
DO NOT STORE IMPORTANT DATA ON THIS DEPLOYMENT

Deployed Images
  • ghcr.io/inovex/scrumlr.io/scrumlr-frontend:sha-28a4012

  • ghcr.io/inovex/scrumlr.io/scrumlr-server:sha-28a4012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inline URLs in detail view
1 participant