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

New linking experience. #17368

Draft
wants to merge 284 commits into
base: master
Choose a base branch
from
Draft

Conversation

pszczesniak
Copy link
Contributor

@pszczesniak pszczesniak commented Oct 31, 2024

Suggested merge commit message (convention)

Type: !!TODO!! describe changes in the link form. Closes #000.

Other (link): The Link feature now uses the ToolbarView instead of a custom LinkActionsView to display the link toolbar in the contextual balloon. The new toolbar uses components registered in the ComponentFactory.

Other (bookmark): The Bookmark feature now uses the WidgetToolbarRepository instead of a custom ActionsView to display the bookmark toolbar in the contextual balloon. The new toolbar uses components registered in the ComponentFactory.

Fix (ui): The ButtonView#icon property can now be set/reset after the button's first render.

Other (ui): The IconView now throws a meaningful error if the provided icon content is not a valid SVG.

Other (widget): The WidgetToolbarRepository#register() now accepts a customized list of desired balloon positions.

BREAKING CHANGE (link): The LinkUI#actionsView is no longer available. The bookmark feature now uses the LinkUI#toolbarView (an instance of the ToolbarView class) instead of the custom LinkActionsView.

BREAKING CHANGE (bookmark): The BookmarkUI#actionsView is no longer available. The Bookmark feature is now using the WidgetToolbarRepository instead.

MINOR BREAKING CHANGE (link): The createBookmarkCallbacks() helper is now replaced by the isScrollableToTarget() and scrollToTarget() helpers.

...
...
...
...

Other (bookmark): Form styles and structure is now unified with use of ck-form and ck-form__row.

Other (image): Image custom resize form styles and structure now unified with use of ck-form and ck-form__row.

Other (link): Form styles and structure is now unified with use of ck-form and ck-form__row.

Other (table): Extracted the form.css to theme-lark package.

Other (table): Extracted the FormRowView to a separate package.

Feature (theme-lark): Introduced the form.css component .

Feature (ui): Introduced the FormRowView class.

MINOR BREAKING CHANGE (table): The FormRowView class was moved to the ui package.

MINOR BREAKING CHANGE (table): The form.css component was moved to the theme-lark package.


Additional information

For example – encountered issues, assumptions you had to make, other affected tickets, etc.

pszczesniak and others added 27 commits December 11, 2024 13:24
# Conflicts:
#	packages/ckeditor5-bookmark/package.json
#	packages/ckeditor5-build-balloon-block/build/ckeditor.js.map
#	packages/ckeditor5-build-balloon/build/ckeditor.js.map
#	packages/ckeditor5-build-classic/build/ckeditor.js.map
#	packages/ckeditor5-build-decoupled-document/build/ckeditor.js.map
#	packages/ckeditor5-build-inline/build/ckeditor.js.map
#	packages/ckeditor5-build-multi-root/build/ckeditor.js.map
#	packages/ckeditor5-link/package.json
…nce-partial-text-update

Feature (link): Formatting is no longer lost after changing a URL or a displayed-text of the selected link.
Other (bookmark): Form styles and structure is now unified with use of `ck-form` and `ck-form__row`.  

Other (image): Image custom resize form styles and structure now unified with use of `ck-form` and `ck-form__row`.  

Other (link): Form styles and structure is now unified with use of `ck-form` and `ck-form__row`.  

Other (table): Extracted the `form.css` to `theme-lark` package.  

Other (table): Extracted the `FormRowView` to a separate package.  

Feature (theme-lark): Introduced the `form.css` component .  

Feature (ui): Introduced the `FormRowView` class.

MINOR BREAKING CHANGE (table): The `FormRowView` class was moved to the `ui` package.  

MINOR BREAKING CHANGE (table): The `form.css` component was moved to the `theme-lark` package.
Fix (link): Should show scroll on link list providers when it contains more than 4 items.
@Mati365
Copy link
Member

Mati365 commented Dec 24, 2024

This issue needs to be double checked on this branch: #17685. I tested it locally, and it looks like problem no longer occurs.

…nce-wrong-tooltips

Remove tooltip from `insert` / `update` link form save button.
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.

6 participants