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

Audit and improves the link component. #1109

Conversation

deepakjosp
Copy link
Contributor

@deepakjosp deepakjosp commented May 24, 2024

Description
Fixes the issues with the link and it's popover component.

Screenshots
Before
image

After
image

Checklist

  • I have made corresponding changes to the documentation.
  • I have updated the types definition of modified exports.
  • I have verified the functionality in some of the neeto web-apps.
  • I have added the necessary label (patch/minor/major - If package publish
    is required).

Reviewers

@neetodeploy neetodeploy bot temporarily deployed to neeto-editor-ou1t-pr-1109 May 24, 2024 10:59 Inactive
@deepakjosp deepakjosp self-assigned this May 24, 2024
@neetodeploy neetodeploy bot temporarily deployed to neeto-editor-ou1t-pr-1109 May 27, 2024 06:03 Inactive
…light the selection when editor was not in focus.
@neetodeploy neetodeploy bot temporarily deployed to neeto-editor-ou1t-pr-1109 May 27, 2024 08:23 Inactive
@deepakjosp deepakjosp linked an issue May 27, 2024 that may be closed by this pull request
5 tasks
@neetodeploy neetodeploy bot temporarily deployed to neeto-editor-ou1t-pr-1109 May 27, 2024 10:42 Inactive
@deepakjosp deepakjosp marked this pull request as ready for review May 27, 2024 11:11
@deepakjosp
Copy link
Contributor Author

@praveen-murali-ind _a please review.

@praveen-murali-ind
Copy link
Contributor

@sreeram-venkitesh _a Could you please check the deployment?

@neetodeploy neetodeploy bot temporarily deployed to neeto-editor-ou1t-pr-1109 May 28, 2024 14:39 Inactive
@neetodeploy neetodeploy bot temporarily deployed to neeto-editor-ou1t-pr-1109 May 28, 2024 14:42 Inactive
@sreeram-venkitesh
Copy link
Member

Health check was failing, I've fixed it and redeployed both the NeetoEditor PRs.

@deepakjosp
Copy link
Contributor Author

@AbhayVAshokan _a please review.

Copy link
Member

@AbhayVAshokan AbhayVAshokan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good. Can you please make sure that these elements are of the same font size? You can merge the PR once done.

@deepakjosp _a:
image

@neetogit-bot neetogit-bot bot assigned deepakjosp and unassigned AbhayVAshokan Jun 3, 2024
@neetodeploy neetodeploy bot requested a deployment to neeto-editor-ou1t-pr-1109 June 4, 2024 11:19 In progress
@deepakjosp deepakjosp merged commit 1e9954d into 1100-neetoeditor-revamp-base-branch Jun 4, 2024
@deepakjosp deepakjosp deleted the 1108-audit-and-improve-the-link-component branch June 4, 2024 11:19
deepakjosp added a commit that referenced this pull request Jun 25, 2024
* Removes slash command plugin from the customExtensions list. (#1107)

* Removed slash command plugin from the customExtensions list.

* Removed the slash command storybook

* Updated version name to beta.

* Updated neetoEditor to v1.34.0-beta1

* Updated version name.

* Incremented the version to publish a release.

* Audit and improves the link component. (#1109)

* Added a subtle animation to the link popover component.

* Replaced the custom border color and box shadow with neetoui helpers and standards that we follow.

* Added a plugin to highlight the selected text when the editor is not in focus.

* Added style for the selected text when the editor is out of focus.

* Optimized and improvised the add link popover position and logic to calculate the position.

* Optimized and fixed issues in removing the decoration applied to highlight the selection when editor was not in focus.

* Added arrrow to the edit/view links popover.

* Optimized the arrow and popover position to be more in sync with the cursor and selection.

* Updated the link text font size and weight to make it consistent with the nearby buttons.

* Updated the version to v1.35.0-beta1

* Implements responsive menu using the three dot approach. (#1118)

* Fixed false result check of the existence of an item in the menu options.

* Moved the popover component to the bottom from out of the menu list since it's a portal.

* Moved the portal components out of the menubar.

* Removed the collapsible menu feature and it's related code and implementations.

* Fixed the layout issue after the collapse feature has been removed.

* Refactored and added necessary props to the menus in the create menu options in font, history, list and block menu groups.

* Moved components to components folder and created Button component.

* Refactored the change in folder location of the fixed menu controls.

* Refactored the Fixed menu to render the menu items from the group of menu returned from the buildMenuItems util.

* Created the MoreMenu to overflowing actions.

* Added logic to separate out menu groups to 2 lists based on the available space.

* Fixed issues in the mention component when moved on to the secondary menu.

* Fixed issues in the emoji component when moved on to the secondary menu.

* Created a common target button for the seondary menu item and fixed the menu position in secondary and primary usages.

* Fixed the UI issues in the text color option when moved to the secondary menu.

* Fixed issues in the TableOption when the option was moved to more menu.

* Fixed issues with the more menu and made it compatible with all the available options.

* Fixed a few UI issues in the menu  buttons and dropdowns.

* Fixed the dependencies list of the buildmenu items handler.

* Fixed the performance issues and the  unexpected behavioural problem with in menu because of the unnecessary dependencies.

* Fixed the isActive and command run issues in the menu buttons and dropdown.

* Moved out the static code from the component to utils.

* Fixed the typo in the filename of SecondaryMenuTarget

* Created a copy of the olde buildMenuItems util for bubble menu since we made significant changes to the buildMenuItems of the fixed menu.

* Removed the unused util to render the menu button.

* Fixed empty tooltip of the font size option dropdown.

* Fixed the empty space on the menubar.

* Removed the appendTo prop and set the strategy fixed on the more dropdown component in fixed menu.

* Reduces the bundle size of attachments preview. (#1125)

* Updated the rollup config to support code splitting.

* Added plugin to cleanup the build output each time.

* Updated the upload files list in the package.json to include dist folder.

* Updated the gitignore to ignore build outputs.

* Lazy loaded the Preview component in the Attachment to prevent react-doc-viewer in the initial chunk of the bundle.

* Removed unnecessary files from the ignore list and replaced the single quotes for strings in the rollup config with double quotes.

* Replaced the singlequotes in the cleaner files list with double quotes.

* Fixed the eager loading of the Preview chunk.

* Updated the version name to beta.

* Set a feedback to the user while fetching the Preview component chunk.

* udpated version name.

* Reset the version change.

* v1.33.4

* Revert "Reduces the bundle size of attachments preview. (#1125)" (#1136)

This reverts commit 5c87e7d.

* v1.33.5

* Improved mentions UI (#1128)

* Improved mentions UI

* Added background color when the mentions item is active

* Made necessary modifications to the special mentions

---------

Co-authored-by: Abhay V Ashokan <abhayvashokan@gmail.com>

* v1.33.6

* Reduces the bundle size of attachments preview.  (#1137)

* Updated the rollup config to support code splitting.

* Added plugin to cleanup the build output each time.

* Updated the upload files list in the package.json to include dist folder.

* Updated the gitignore to ignore build outputs.

* Lazy loaded the Preview component in the Attachment to prevent react-doc-viewer in the initial chunk of the bundle.

* Removed unnecessary files from the ignore list and replaced the single quotes for strings in the rollup config with double quotes.

* Replaced the singlequotes in the cleaner files list with double quotes.

* Fixed the eager loading of the Preview chunk.

* Updated the version name to beta.

* Set a feedback to the user while fetching the Preview component chunk.

* udpated version name.

* Reset the version change.

* Moves common dependencies to the peer list. (#1129)

* Moved commons dependencies to the peer list.

* Added rollup plugin visualizer.

* v1.34.0

---------

Co-authored-by: Abhay V Ashokan <abhay.ashokan@bigbinary.com>
Co-authored-by: Abhay V Ashokan <35297280+AbhayVAshokan@users.noreply.github.com>
Co-authored-by: Praveen Murali <itspraveenmurali@gmail.com>
Co-authored-by: Abhay V Ashokan <abhayvashokan@gmail.com>

* Updated version to v1.35.1-beta1

* Updated version to v1.36.0-beta1

* Updates all tiptap packages to v2.4.0 (#1145)

* Updated @tiptap/extension-bubble-menu to 2.4.0 from 2.0.0-beta.199

* Updated all tiptap packages to 2.4.0

* Fixed the focus issue when trying to edit a link.

* Checked in the change minified CSS output.

* Reverted the change in edit link title since the bug was fixed in the tiptap release.

* Updated editor version to v1.36.2-beta1

* Replaced the resize event listener with ResizeObserver

---------

Co-authored-by: Abhay V Ashokan <abhayvashokan@gmail.com>
Co-authored-by: Abhay V Ashokan <abhay.ashokan@bigbinary.com>
Co-authored-by: Abhay V Ashokan <35297280+AbhayVAshokan@users.noreply.github.com>
Co-authored-by: Praveen Murali <itspraveenmurali@gmail.com>
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.

Audit and improve the Link component.
5 participants