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

add an article about notifications #10

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from
Draft

Conversation

albertotirla
Copy link
Member

because of the current sprint of development, we have a new feature, reading notifications. This article talks about that
the commit was there originally on main, but now I'm doing some hacks to allow this to be reviewed properly before making it available to the public

@albertotirla
Copy link
Member Author

alright, this is almost ready for merging, if you resolve the threads I attached to the file. Now, one more thing, I recommend you keep the conclusion as it was, because the so called fluf is important there, it's what characterizes our blog in a way, how we really show the reader that they are important to the project.

@TTWNO
Copy link
Member

TTWNO commented Mar 31, 2024

I don't see the threads you are talking about?

@albertotirla
Copy link
Member Author

I don't see the threads you are talking about?

that's extremely weird, this happens on desktop too?

Copy link
Member Author

@albertotirla albertotirla left a comment

Choose a reason for hiding this comment

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

alright, I left some feedback regarding this pr

ever since the [first release]({{<relref "release_0-1-0">}}), things appeared to be quite stagnant, especially from the outside. This is not an imagination of the public, this is actually true, things were quite stagnant for a while.
This is because, one of our maintainers, Tait, has something to say about that:

> I thought it would be easy, or at least not take very long, so I just went for it.
Copy link
Member Author

Choose a reason for hiding this comment

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

this is a bit too abrupt, people don't know what you're talking about, so before you do that, a bit of explanation would be required

> I thought it would be easy, or at least not take very long, so I just went for it.
> I spent probably a few thousand hours trying to figure out:
> - Where is the right place for this protocol (DBus spec, Wayland extention)
> - How would the specification bridge to other protocols?
Copy link
Member Author

Choose a reason for hiding this comment

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

what protocols? most of the readers don't know what that is

## Diving Into The Hacks

In response to the absence of an accessibility dedicated API for system information in the freedesktop stack, various hacks have emerged over time.
Our approach, while different in to Orca, isn't too far off from existing solutions.
Copy link
Member Author

Choose a reason for hiding this comment

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

different from orca, not into. I would make it a suggestion, but I dk how, so it's a line comment

### Odilia's Hack

Odilia's hack relies on the [freedesktop notifications specification](https://specifications.freedesktop.org/notification-spec/notification-spec-latest.html) to be implemented by notification daemons.
This required some up-front design work to essentially write our own notification daemon, but this will be extremely resilient to long-term change.
Copy link
Member Author

Choose a reason for hiding this comment

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

not at all, we didn't write our own notification daemon. Instead, we used monitoring to sniff the bus for calls to the methods which are used to post a notification to the existing daemon. We can't make a daemon, I think, because from what I read, having more than one notification daemon in the system means that one of them will get the signals, while the other won't, so odilia might read the notification, but it might not be displayed on the desktop environment's place for notifications

Copy link
Member

Choose a reason for hiding this comment

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

Yep, whoops.

content/news/notifications.md Show resolved Hide resolved

This approach has its own advantages:

1. We don't rely on presentation. Each notification daemon can present things however it wants on screen, as long as it receives notification through the standard notification specification.
Copy link
Member Author

Choose a reason for hiding this comment

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

either recieves a notification, or recieves notifications

@TTWNO TTWNO marked this pull request as draft May 15, 2024 16:23
@TTWNO
Copy link
Member

TTWNO commented May 15, 2024

@albertotirla please check changes

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.

2 participants