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

Correct misleading description of SX action #1886

Merged
merged 2 commits into from
Mar 17, 2024
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions gnucash/ui/gnc-plugin-page-register.ui
Original file line number Diff line number Diff line change
Expand Up @@ -294,7 +294,7 @@
<item>
<attribute name="label" translatable="yes">Sche_dule…</attribute>
<attribute name="action">GncPluginPageRegisterActions.ScheduleTransactionAction</attribute>
<attribute name="tooltip" translatable="yes">Create a Scheduled Transaction with the current transaction as a template</attribute>
<attribute name="tooltip" translatable="yes">Create a Scheduled Transaction with the current transaction as a template or edit the Scheduled Transaction that current transaction was created by</attribute>
Copy link
Member

Choose a reason for hiding this comment

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

Please finish the sentence with a full stop .!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Are you sure? I know the grammar, but there are no full stops at the end of strings/sentences in this file (except for the line 99).

Copy link
Member

Choose a reason for hiding this comment

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

It would be nice, if you fixed all then.
In the times of DOS almost nobody applied them, but today weblate [old message] is sending us warnings.

<attribute name="temp" translatable="no">yes</attribute>
</item>
<item>
Expand Down Expand Up @@ -431,7 +431,7 @@
<item>
<attribute name="label" translatable="yes">Sche_dule…</attribute>
<attribute name="action">GncPluginPageRegisterActions.ScheduleTransactionAction</attribute>
<attribute name="tooltip" translatable="yes">Create a Scheduled Transaction with the current transaction as a template</attribute>
<attribute name="tooltip" translatable="yes">Create a Scheduled Transaction with the current transaction as a template or edit the Scheduled Transaction that current transaction was created by</attribute>
</item>
<item>
<attribute name="label" translatable="yes">_Jump to the other account</attribute>
Expand Down Expand Up @@ -543,7 +543,7 @@
<item>
<attribute name="label" translatable="yes">Sche_dule…</attribute>
<attribute name="action">GncPluginPageRegisterActions.ScheduleTransactionAction</attribute>
<attribute name="tooltip" translatable="yes">Create a Scheduled Transaction with the current transaction as a template</attribute>
<attribute name="tooltip" translatable="yes">Create a Scheduled Transaction with the current transaction as a template or edit the Scheduled Transaction that current transaction was created by</attribute>
</item>
<item>
<attribute name="label" translatable="yes">_Jump to the other account</attribute>
Expand Down Expand Up @@ -846,7 +846,7 @@
<property name="can-focus">False</property>
<property name="label" translatable="yes">Sche_dule…</property>
<property name="action-name">GncPluginPageRegisterActions.ScheduleTransactionAction</property>
<property name="tooltip-text" translatable="yes">Create a Scheduled Transaction with the current transaction as a template</property>
<property name="tooltip-text" translatable="yes">Create a Scheduled Transaction with the current transaction as a template or edit the Scheduled Transaction that current transaction was created by</property>
Copy link
Member

Choose a reason for hiding this comment

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

Full stop still missing.
I will fix it despite string freeze, because it will be the same string as we already have.

Copy link
Contributor Author

@Kaligula0 Kaligula0 Mar 24, 2024

Choose a reason for hiding this comment

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

Sorry! My Regex didn't fix all of them, because I didn't notice there are <attributes> as well as <properties>. Now I found even more full stops, and fixed them all, I think → #1895.

<property name="use-underline">True</property>
<property name="icon-name">gnc-sx-new</property>
</object>
Expand Down
Loading