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: 24 see components from sidebar #68

Merged
merged 15 commits into from
Jan 5, 2025
Merged

Conversation

egenerse
Copy link
Collaborator

@egenerse egenerse commented Dec 30, 2024

Checklist

  • I linked PR with a related issue
  • I added multiple screenshots/screencasts of my UI changes

Motivation and Context

This PR completes #24

Description

Sidebar is added and sidebar elements can be added into canvas
SVGs folder is created for reusability to render elements in Canvas and also in Sidebar

Steps for Testing

  1. Build project with latest changes
  2. Run project
  3. Drag and drop elements from sidebar into canvas
  4. Check if elements are created on Canvas

Screenshots

safari chrome

safariVideo

video2

@egenerse egenerse self-assigned this Dec 30, 2024
@egenerse egenerse linked an issue Dec 30, 2024 that may be closed by this pull request
Copy link
Collaborator

@kurunbelemir kurunbelemir left a comment

Choose a reason for hiding this comment

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

Color description is missing. Can you add it?

Copy link
Collaborator

@kurunbelemir kurunbelemir left a comment

Choose a reason for hiding this comment

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

image Sidebar looks buggy in Safari? Can we fix this? Also drag and drop does not work like in Google Chrome.

Copy link
Collaborator

@kurunbelemir kurunbelemir left a comment

Choose a reason for hiding this comment

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

Screenshot 2024-12-30 at 23 25 37 Also Color Description looks odd. When you click for dragging the element, It creates bigger outer box. Then when you drop, color description element size gets bigger. When I checked current Apollon version, we need to make smaller color description elements like you displayed in the canvas.

@egenerse
Copy link
Collaborator Author

Screenshot 2024-12-30 at 23 25 37 Also Color Description looks odd. When you click for dragging the element, It creates bigger outer box. Then when you drop, color description element size gets bigger. When I checked current Apollon version, we need to make smaller color description elements like you displayed in the canvas.

I saw these 2 bugs and fixed but didnt push. I am looking into the safari bug now

Copy link
Collaborator

@kurunbelemir kurunbelemir left a comment

Choose a reason for hiding this comment

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

ApollonDragandDropBug.mp4

Is this drag and drop correct? It looks little jumpy to me. After getting elements from sidebar, changing placement in the canvas works perfectly. Only the first drag and drop looks weird.
Also Left edges of the elements looks grey while other edges are black. Do you know the reason?

@egenerse egenerse requested a review from kurunbelemir January 5, 2025 09:33
Copy link
Collaborator

@kurunbelemir kurunbelemir left a comment

Choose a reason for hiding this comment

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

Works perfect and looks good.

@egenerse egenerse merged commit fb9ed07 into main Jan 5, 2025
2 checks passed
@egenerse egenerse deleted the 24-see-components-from-sidebar branch January 5, 2025 16:02
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.

See Components from Sidebar
2 participants