Added New Window Prompt

[rakshit087]

Fixes #155

Edited the desktop file to add a 'New Window' prompt when user right clicks.

[jeremypw]

As a hint, its better to create a new branch on your fork and modify that.

[rakshit087]

Okay I will create a new branch, or shall I remove %U in both the execution commands?

[jeremypw]

Okay I will create a new branch, or shall I remove %U in both the execution commands?

I am just checking that everything works in Flatpak when those parameters are removed. Calculator is a bit unusual - most elementary apps are singletons by default and you have to apply an option in order for a new window to open. However, its best not to allow mission drift for one PR. If other changes that are not essential to this PR are desired they should be made in another PR.

[rakshit087]

Okay! Let me know if everything works if those parameters are removed, I will make the necessary changes 😁

PS: I am actually new to open-source space, so I don't know how can I change the branch of this push request. I will do so after learning about it 😅

[jeremypw]

I have now confirmed that installing as a Flatpak with the %U removed both in Exec and in the NewWindow action works OK. The new action appears in the dock and in the Applications Menu as expected. So in this case I think we can fix both issues at once - thanks!

[rakshit087]

Wow! Thank you so much will make a new commit in a min.
Commited :))

[jeremypw]

Looks good now - but just remembered that you should update the appdata release information when changes significantly impact the UX.

[rakshit087]

Okay! How can I do so? Sorry I am kind of new to opensource and stuff. Shall I make it 1.7.1?

[jeremypw]

You need to edit /data/io.elementary.calculator.appdata.xml.in and add another <li></li> entry to release 1.7.1.

Something like <li>Add New Window Desktop Action</li>

[rakshit087]

Got it, working on it

[rakshit087]

Done 👍

[jeremypw]

Looks good to go - thanks!

[rakshit087]

As a hint, its better to create a new branch on your fork and modify that.

I finally understand this. Will keep this in mind when I make a PR next time 👍
Thank you for merging.