Skip to content

Remove inserted element position #1733

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

Merged
merged 1 commit into from
Apr 4, 2025
Merged

Remove inserted element position #1733

merged 1 commit into from
Apr 4, 2025

Conversation

Kitenite
Copy link
Contributor

@Kitenite Kitenite commented Apr 4, 2025

Description

Related Issues

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Release
  • Refactor
  • Other (please describe):

Testing

Screenshots (if applicable)

Additional Notes


Important

Removes position style from inserted elements in InsertManager across two files to fix positioning issues.

  • Behavior:
    • Removes position: 'relative' from default properties in getDefaultProperties() for EditorMode.INSERT_DIV in InsertManager.
    • Removes position: 'absolute' from styles in createInsertAction() for non-text elements in InsertManager.
  • Files Affected:
    • apps/studio/src/lib/editor/engine/insert/index.ts
    • apps/web/src/components/store/editor/engine/insert/index.ts

This description was created by Ellipsis for 5251fc0. It will automatically update as commits are pushed.

Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 5251fc0 in 1 minute and 7 seconds

More details
  • Looked at 64 lines of code in 2 files
  • Skipped 0 files when reviewing.
  • Skipped posting 8 drafted comments based on config settings.
1. apps/studio/src/lib/editor/engine/insert/index.ts:42
  • Draft comment:
    Removed 'position' property from default DIV styling. Ensure that removal doesn't break layout.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
2. apps/web/src/components/store/editor/engine/insert/index.ts:42
  • Draft comment:
    Removed explicit 'position' property from DIV styling. Confirm that element positioning works as expected without it.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
3. apps/web/src/components/store/editor/engine/insert/index.ts:1
  • Draft comment:
    PR description mentions adding onboarding docs, but changes here only remove CSS position styling. Confirm that PR description is updated to reflect the actual changes.
  • Reason this comment was not posted:
    Comment was not on a location in the diff, so it can't be submitted as a review comment.
4. apps/studio/src/lib/editor/engine/insert/index.ts:44
  • Draft comment:
    Removed 'position: relative' from DIV default styles. Ensure layout is correctly handled elsewhere.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
5. apps/studio/src/lib/editor/engine/insert/index.ts:180
  • Draft comment:
    Removed 'position: absolute' from insertion styles. Confirm that positioning is now managed externally.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
6. apps/web/src/components/store/editor/engine/insert/index.ts:43
  • Draft comment:
    Removed 'position: absolute' from default DIV styles. Verify that this change maintains the intended element positioning.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
7. apps/web/src/components/store/editor/engine/insert/index.ts:179
  • Draft comment:
    Removed 'position: absolute' from the styles in createInsertAction; ensure that element placement remains as expected.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None
8. apps/web/src/components/store/editor/engine/insert/index.ts:24
  • Draft comment:
    Constructor formatting changed (extra space added). Please ensure consistency with project style guidelines.
  • Reason this comment was not posted:
    Confidence changes required: 33% <= threshold 50%
    None

Workflow ID: wflow_tYS696Y8mFA5VTJX


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@Kitenite Kitenite merged commit 2f1f86a into main Apr 4, 2025
@Kitenite Kitenite deleted the bugs/relative-position branch April 4, 2025 22:01
ml-delaurier pushed a commit to ml-delaurier/nolook that referenced this pull request Apr 23, 2025
t1c1 pushed a commit to t1c1/onlookbotcodes that referenced this pull request Jun 5, 2025
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.

1 participant