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

"+" icon in List Editor should be finished, as it is easy now to confuse it with a mouse cursor. #6517

Closed
2 tasks
sylwiabr opened this issue May 3, 2023 · 5 comments
Assignees
Labels
d-unknown Difficulty: unable to estimate difficulty p-highest Should be completed ASAP s-invalid Status: not valid for some reason

Comments

@sylwiabr
Copy link
Member

sylwiabr commented May 3, 2023

Discord username

No response

What type of issue is this?

Permanent – Occurring repeatably

Is this issue blocking you from using Enso?

  • Yes, I can't use Enso because of this issue.

Is this a regression?

  • Yes, previous version of Enso did not have this issue.

What issue are you facing?

Add a new node, try to move it - the cursor pointer will be rendered on the node and never disappear
image

Expected behaviour

I should have just one cursor

How we can reproduce it?

No response

Screenshots or screencasts

No response

Enso Version

Nightly 03.05

Browser or standalone distribution

Standalone distribution

Browser Version or standalone distribution

Standalone

Operating System

MacOS

Operating System Version

No response

Hardware you are using

No response

@sylwiabr sylwiabr added --bug Type: bug triage labels May 3, 2023
@github-project-automation github-project-automation bot moved this to ❓New in Issues Board May 3, 2023
@sylwiabr sylwiabr added -gui --regression Important: regression p-highest Should be completed ASAP labels May 3, 2023
@sylwiabr sylwiabr moved this from ❓New to 📤 Backlog in Issues Board May 3, 2023
@sylwiabr sylwiabr added this to the Design Partners milestone May 3, 2023
@sylwiabr sylwiabr moved this from 📤 Backlog to ❓New in Issues Board May 3, 2023
@sylwiabr
Copy link
Member Author

sylwiabr commented May 3, 2023

This is happening also on Windows

@vitvakatu
Copy link
Contributor

It's quite easy to reproduce - add a .split node for text literal input. It seems the cursor (or a shape that is similar to the cursor) is displayed in place of the array literal. Clicking on it creates a new [_] placeholder. So it has to be connected with the vector editor implementation.

@vitvakatu
Copy link
Contributor

I don't think it makes sense to bisect it, it is rather clear that it came together with vector editor, and we don't want to rollback any of the related PRs anyway. So I'm removing regression label.

@vitvakatu vitvakatu added d-unknown Difficulty: unable to estimate difficulty g-widgets and removed --regression Important: regression triage labels May 3, 2023
@wdanilo
Copy link
Member

wdanilo commented May 3, 2023

This is not a cursor duplication. The circle you see on the node is something we did not finish - it would be a "+" icon telling you that you can add new items there, it is not a cursor. It is displayed on empty vector editor. We need to finish it and display + icon there instead.

@wdanilo wdanilo removed the --bug Type: bug label May 3, 2023
@wdanilo wdanilo changed the title Cursor stick to the node and never disappear "+" icon in List Editor should be finished, as it is easy now to confuse it with a mouse cursor. May 3, 2023
@sylwiabr sylwiabr moved this from ❓New to 📤 Backlog in Issues Board May 3, 2023
@vitvakatu vitvakatu moved this from 📤 Backlog to 🔧 Implementation in Issues Board May 12, 2023
@farmaazon
Copy link
Contributor

As #6540 removes the icon for some time (until it will be restored when implementing the design), I think the task is no longer valid. For future reference, the shape definition is something along:

mod add_element_icon {
    use super::*;

    ensogl_core::shape! {
        (style: Style, color_rgba: Vector4) {
            let size = Var::canvas_size();
            let radius = Min::min(size.x(),size.y()) / 2.0;
            let circle = Circle(&radius);
            let bg = circle.fill(INVISIBLE_HOVER_COLOR);

            let angle= Radians::from(90.0.degrees());
            let bar_length= &radius * 5.0 / 4.0;
            let bar_width= &bar_length / 5.0;
            #[allow(clippy::disallowed_names)] // The `bar` name here is totally legit.
            let bar= Rect((bar_length, &bar_width));
            let plus = bar.rotate(angle) + bar;
            let shape= circle - plus;
            shape.fill(color_rgba);
            (shape + bg).into()
        }
    }
}

May be further improved by taking plus size and color from styles.

@farmaazon farmaazon closed this as not planned Won't fix, can't repro, duplicate, stale May 12, 2023
@github-project-automation github-project-automation bot moved this from 🔧 Implementation to 🟢 Accepted in Issues Board May 12, 2023
@farmaazon farmaazon added the s-invalid Status: not valid for some reason label May 12, 2023
@farmaazon farmaazon moved this from 🟢 Accepted to 🗄️ Archived in Issues Board May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
d-unknown Difficulty: unable to estimate difficulty p-highest Should be completed ASAP s-invalid Status: not valid for some reason
Projects
Archived in project
Development

No branches or pull requests

4 participants