mirror of
https://github.com/kusti8/proton-native.git
synced 2026-05-15 14:15:50 -06:00
[GH-ISSUE #127] Feature: Add native notifications support #78
Labels
No labels
bug
documentation
enhancement
libui issue
pull-request
question
wait for libui implementation
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/proton-native#78
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @bardiarastin on GitHub (May 14, 2018).
Original GitHub issue: https://github.com/kusti8/proton-native/issues/127
Hello, awesome project 👏👏👏
I was able to add native OS notifications using https://github.com/mikaelbr/node-notifier
I'm thinking of that we can add this lib to project to have built in notification support simply by bridging.
I can make a PR for this if everything is okay with adding support for notifications, along side this, I have a lot of other ideas and things missing in this project like lack of support for printing for example...
@mischnic commented on GitHub (May 14, 2018):
Proton-native is based on libui, there is an issue for notifications there (https://github.com/andlabs/libui/issues/338), but it's not going to happen very soon (if ever): "I still don't know if this is in scope for libui."
How should that work? What would you like to print? (PDFs, images?)
@bardiarastin commented on GitHub (May 14, 2018):
@mischnic
I taught that libui is only for UI stuff and nothing more so we should consider adding other extra stuff ourselves.
please let me know if you decided to do it in proton native I have ideas for performance of this (electron app with 3000 daily active users).
for printing there are some several functionalities needed. like a method to return OS printers, a method to print with options like which printer, page size, adjusting margins and...
what we needed was printing an html document in electron which we could make a browser window put html into that with whatever css we wanted then print it, along side support for image printing, I can make a separate issue for printing.
@mischnic commented on GitHub (May 14, 2018):
I would say that notifications are part of the ui.
With my previous comment I wanted to say: it should be done this way because it won't happen in libui.
What keeps you from using that library together with proton-native as it is?
What about https://github.com/tojocky/node-printer ? (Haven't tested it myself)
@kusti8 commented on GitHub (May 14, 2018):
If there's no special thing that needs to be done on Proton Native to make it work, it's best if it's not merged. This leads to additional and additional libraries being merged just bloating the project. I can mention it in the documentation, but I'd rather not add another library.