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
freeoffice: init at 966 #67105
freeoffice: init at 966 #67105
Conversation
mimeType = "application/x-pmd;application/x-pmdx;application/x-pmv;application/excel;application/x-excel;application/x-ms-excel;application/x-msexcel;application/x-sylk;application/x-xls;application/xls;application/vnd.ms-excel;application/vnd.stardivision.calc;application/vnd.openxmlformats-officedocument.spreadsheetml.sheet;application/vnd.openxmlformats-officedocument.spreadsheetml.template;application/vnd.ms-excel.sheet.macroenabled.12;application/vnd.openxmlformats-officedocument.spreadsheetml.template;application/vnd.ms-excel.template.macroEnabled.12;application/x-dif;text/spreadsheet;text/csv;application/x-prn;application/vnd.ms-excel.sheet.binary.macroenabled.12"; | ||
extraEntries = '' | ||
TryExec=${pname}-planmaker | ||
StartupWMClass=tm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do all the apps really have the same WM_CLASS?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch! This was a copy & paste error. Fixed now.
a3f4b98
to
3a67843
Compare
Do the wm classes have to be two-letter combos? Is there a reason to not name them, for example, Or, maybe prefix the class names with |
They need to match the |
Yep, you're right. |
Also, I've played around with this expression, and in terms of functionality, it LGTM. |
Yep, unfortunately, they have decided to use these two-letter names. Oh well, as long as everyone else isn't ;). |
@jtojnar I hope I have addressed all your comments. Would be nice to have this in 19.09. |
FreeOffice is a freeware office suite consisting of: - A word processor: TextMaker - A spreadsheet: PlanMaker - A presentation package: Presenter FreeOffice is a stripped down version of SoftMaker Office, which is not freeware.
3a67843
to
760fe22
Compare
It turns out that it is easy to add Softmaker Office as well, while sharing a large part of the derivation. I will create a new PR adding both FreeOffice and Softmaker Office. |
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/prs-ready-for-review-may-2019/3032/54 |
Motivation for this change
FreeOffice is a freeware office suite consisting of:
FreeOffice is a stripped down version of SoftMaker Office, which is not
freeware.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @