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(rich-text-editor): DLT-2237 add image extension to editor #589

Merged
merged 2 commits into from
Dec 11, 2024

Conversation

emariotti3
Copy link
Contributor

@emariotti3 emariotti3 commented Dec 10, 2024

feat(rich-text-editor): DLT-2237 add image extension to editor

Obligatory GIF (super important!)

Obligatory GIF

🛠️ Type Of Change

These types will increment the version number on release:

  • Fix
  • Feature
  • Performance Improvement
  • Refactor

📖 Jira Ticket

DLT-2237

📖 Description

Added image extension to editor in order to be able to render images inside the editor.

For us to be able to support this we need to add the tiptap image extension to the editor.

💡 Context

We need this because as part of our implementation of the ‘forward’ feature for email, we want to add quotes to previous emails in the editor, and allow the user to edit the forwarded emails as desired.

However, the emails can contain inline images in the email body or as part of the email signatures, which would be included in the quotes.

📝 Checklist

For all PRs:

  • I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
  • I have reviewed my changes.
  • I have added all relevant documentation.
  • I have considered the performance impact of my change.

For all Vue changes:

  • I have added / updated unit tests.
  • I have made my changes in Vue 2 and Vue 3. Note: you may sync your changes from Vue 2 to Vue 3 (or vice versa) using the ./scripts/dialtone-vue-sync.sh script. Read docs here: Dialtone Vue Sync Script
  • I have validated components with a screen reader.
  • I have validated components keyboard navigation.

📷 Screenshots / GIFs

Screenshot 2024-12-10 at 6 57 39 PM

🔗 Sources

tiptap image extension

@emariotti3 emariotti3 force-pushed the DLT-2237-add-image-extension-to-editor branch from b13ce9f to 2e64ff5 Compare December 10, 2024 22:44
@emariotti3 emariotti3 changed the title DLT-2237: add image extension to editor feat(rich-text-editor): DLT-2237 add image extension to editor Dec 10, 2024
@braddialpad
Copy link
Contributor

merge in staging to fix the accessibility test issue.

@emariotti3 emariotti3 force-pushed the DLT-2237-add-image-extension-to-editor branch from 14ab8d7 to c3699a4 Compare December 10, 2024 23:30
Copy link

Please add either the visual-test-ready or no-visual-test label to this PR depending on whether you want to run visual tests or not.
It is recommended to run visual tests if your PR changes any UI. ‼️

Copy link

✔️ Deploy previews ready!
😎 Dialtone preview: https://dialtone.dialpad.com/deploy-previews/pr-589/
😎 Dialtone-vue 2 preview: https://dialtone.dialpad.com/vue/deploy-previews/pr-589/
😎 Dialtone-vue 3 the preview: https://dialtone.dialpad.com/vue3/deploy-previews/pr-589/

@emariotti3 emariotti3 merged commit 72cff78 into staging Dec 11, 2024
10 checks passed
@emariotti3 emariotti3 deleted the DLT-2237-add-image-extension-to-editor branch December 11, 2024 03:10
braddialpad pushed a commit that referenced this pull request Dec 11, 2024
Co-authored-by: Eugenia Mariotti <eugenia.mariotti@dialpad.com>
braddialpad pushed a commit that referenced this pull request Dec 13, 2024
# [9.87.0](dialtone/v9.86.2...dialtone/v9.87.0) (2024-12-12)

### Bug Fixes

* recipes css cleanup ([#581](#581)) ([ba85404](ba85404))
* **Rich Text Editor:** DP-116205 fix line break related issues ([#573](#573)) ([a204fd4](a204fd4))

### Features

* **Callbar Button:** DLT-2211 add passthrough tooltip props ([#578](#578)) ([66cf439](66cf439))
* **Emoji:** NO-JIRA update emoji-toolkit to 9.0 ([#587](#587)) ([9684377](9684377))
* **Rich Text Editor:** DLT-2237 add image extension to editor ([#589](#589)) ([72cff78](72cff78))
braddialpad pushed a commit that referenced this pull request Dec 13, 2024
# [2.168.0](dialtone-vue2/v2.167.1...dialtone-vue2/v2.168.0) (2024-12-12)

### Bug Fixes

* recipes css cleanup ([#581](#581)) ([ba85404](ba85404))
* **Rich Text Editor:** DP-116205 fix line break related issues ([#573](#573)) ([a204fd4](a204fd4))

### Features

* **Callbar Button:** DLT-2211 add passthrough tooltip props ([#578](#578)) ([66cf439](66cf439))
* **Emoji:** NO-JIRA update emoji-toolkit to 9.0 ([#587](#587)) ([9684377](9684377))
* **Rich Text Editor:** DLT-2237 add image extension to editor ([#589](#589)) ([72cff78](72cff78))
braddialpad pushed a commit that referenced this pull request Dec 13, 2024
# [3.161.0](dialtone-vue3/v3.160.1...dialtone-vue3/v3.161.0) (2024-12-12)

### Bug Fixes

* recipes css cleanup ([#581](#581)) ([ba85404](ba85404))
* **Rich Text Editor:** DP-116205 fix line break related issues ([#573](#573)) ([a204fd4](a204fd4))

### Features

* **Callbar Button:** DLT-2211 add passthrough tooltip props ([#578](#578)) ([66cf439](66cf439))
* **Emoji:** NO-JIRA update emoji-toolkit to 9.0 ([#587](#587)) ([9684377](9684377))
* **Rich Text Editor:** DLT-2237 add image extension to editor ([#589](#589)) ([72cff78](72cff78))
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.

3 participants