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

Re-add a Camera3D icon gizmo to the 3D editor, renew Camera2D/3D icons #53104

Merged

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Sep 26, 2021

Marked as draft, as new Camera2D and Camera3D node icons should be created to match the new icon gizmo design.

The lack of icon gizmo made it no longer possible to select Camera3D nodes in the 3D viewport by clicking on them.

A new Camera3D icon gizmo was designed to be direction-neutral. This avoids the issue that the previous icon gizmo had, since it was always facing right.

This closes #53083.

Preview

Before After
Screenshot_20230808_155324 Screenshot_20230808_155431
Before After
Screenshot_20230808_155334 Screenshot_20230808_160346

Listener3D on the left, Camera3D in the middle, ReflectionProbe on the right.

image

@reduz
Copy link
Member

reduz commented Jul 31, 2022

Looks good to me, what is it missing?

@Calinou
Copy link
Member Author

Calinou commented Jul 31, 2022

Looks good to me, what is it missing?

As I wrote in OP:

Marked as draft, as new Camera2D and Camera3D node icons should be created to match the new icon gizmo design.

I've started working on those icons a while ago, I'll try to finish them ASAP.

@Calinou Calinou force-pushed the editor-readd-camera-gizmo-icon branch from 286f451 to 3364211 Compare July 31, 2022 22:39
@Calinou Calinou changed the title Readd a Camera3D icon gizmo to the 3D editor Readd a Camera3D icon gizmo to the 3D editor, renew Camera2D/3D icons Jul 31, 2022
@Calinou
Copy link
Member Author

Calinou commented Jul 31, 2022

I pushed new icons for Camera2D and Camera3D:

2022-08-01_00 38 36

Let me know if this is OK. CameraEffects CameraAttributes, CameraTexture, ClippedCamera3D and XRCamera3D icons still need to be changed.

Copy link
Member

@adamscott adamscott left a comment

Choose a reason for hiding this comment

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

Seems good to me.

@YuriSizov YuriSizov modified the milestones: 4.0, 4.1 Feb 10, 2023
@YuriSizov YuriSizov modified the milestones: 4.1, 4.x Jun 14, 2023
@akien-mga
Copy link
Member

It's been in draft for 2 years, maybe we can merge what's made already? Or poke MewPurPur if you need help ;)

The lack of icon gizmo made it no longer possible to select Camera3D
nodes in the 3D viewport by clicking on them.

A new Camera3D icon gizmo was designed to be direction-neutral.
This avoids the issue that the previous icon gizmo had, since it was
always facing right.

This also adds icons for CameraAttributesPractical and CameraAttributesPhysical.
@Calinou Calinou force-pushed the editor-readd-camera-gizmo-icon branch from 3364211 to e21388d Compare August 8, 2023 14:05
@Calinou Calinou marked this pull request as ready for review August 8, 2023 14:05
@Calinou Calinou requested a review from a team as a code owner August 8, 2023 14:05
@Calinou
Copy link
Member Author

Calinou commented Aug 8, 2023

It's been in draft for 2 years, maybe we can merge what's made already? Or poke MewPurPur if you need help ;)

I've finished all icons; it's ready for review now.

@akien-mga akien-mga modified the milestones: 4.x, 4.2 Aug 8, 2023
@akien-mga akien-mga merged commit b8d5219 into godotengine:master Aug 8, 2023
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@Calinou Calinou deleted the editor-readd-camera-gizmo-icon branch August 12, 2023 08:39
@akien-mga akien-mga changed the title Readd a Camera3D icon gizmo to the 3D editor, renew Camera2D/3D icons Re-add a Camera3D icon gizmo to the 3D editor, renew Camera2D/3D icons Nov 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spatial camera unselectable in editor and icon gizmo not shown
5 participants