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

Notification list view. #5728

Closed
farmaazon opened this issue Feb 22, 2023 · 1 comment
Closed

Notification list view. #5728

farmaazon opened this issue Feb 22, 2023 · 1 comment
Assignees
Labels
d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints s-wontfix Status: will not be fixed / not a bug x-new-feature Type: new feature request
Milestone

Comments

@farmaazon
Copy link
Contributor

farmaazon commented Feb 22, 2023

What should work from the user perspective

On the right-top corner of the scene, there should be a bell icon: clicking on it should show the notification list. The notifications should be HTML boxes with a short message, displayed along the right edge of the scene, sorted from newest to oldest. User should be able to remove notification from the list. Clicking again should hide the notification list.

There are four kinds of notifications: process, event, error, and warnings. Notifications of different kinds are displayed with different background color. Processes are removed from the list when some process is finished (e.g. compiling stdlib) without user intervention.

If a new notification arrives and the list is hidden, the bell icon should be in "unread" state. This state is removed once the user will show the notification list.

The user should see the notifications we currently display in the "status bar" (i.e. warning about unsupported engine version, compiling standard library process, error about losing connection).

The look of the notifications can be quite plain, there is another task for improving it. Scrolling notification is also not in the scope of this task.

How it should be implemented

The API for controllers to push new notifications should be like those in StatusBarView, but with new notification kinds.

As the EnsoGL text area does not handle multiline text well, the notification boxes should be displayed as DOM divs.
Only the visible notifications shall be rendered.

@farmaazon farmaazon self-assigned this Feb 22, 2023
@farmaazon farmaazon added -gui d-intermediate Difficulty: some prior knowledge required x-new-feature Type: new feature request labels Feb 22, 2023
@farmaazon farmaazon added this to the Beta Release milestone Feb 22, 2023
@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Feb 22, 2023
@farmaazon farmaazon added the p-medium Should be completed in the next few sprints label Feb 24, 2023
@farmaazon farmaazon moved this from 📤 Backlog to ❓New in Issues Board Jul 3, 2023
@farmaazon farmaazon moved this from ❓New to 📤 Backlog in Issues Board Jul 31, 2023
@farmaazon farmaazon removed this from Issues Board Aug 7, 2023
@farmaazon farmaazon added the s-wontfix Status: will not be fixed / not a bug label Mar 4, 2024
@farmaazon
Copy link
Contributor Author

farmaazon commented Mar 4, 2024

The tasks for #5201 should be redesigned for new GUI

@farmaazon farmaazon closed this as not planned Won't fix, can't repro, duplicate, stale Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
d-intermediate Difficulty: some prior knowledge required p-medium Should be completed in the next few sprints s-wontfix Status: will not be fixed / not a bug x-new-feature Type: new feature request
Projects
None yet
Development

No branches or pull requests

3 participants