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

projectorganizer: Use the word "symbol" instead of "tag" #351

Merged
merged 2 commits into from
Feb 18, 2016

Conversation

techee
Copy link
Member

@techee techee commented Feb 17, 2016

Do the same for ProjectOrganizer as for Geany in geany/geany#582

@@ -620,7 +620,7 @@ gint prjorg_project_add_properties_tab(GtkWidget *notebook)
gtk_entry_set_text(GTK_ENTRY(e->ignored_dirs_patterns), str);
g_free(str);

label = gtk_label_new(_("Generate tags for all project files:"));
label = gtk_label_new(_("Parse symbols for all project files:"));
Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure if the new sentence is correct. I'd rather say Parse symbols in all project files or from rather than for. @elextr ?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, and actually we don't want to "parse symbols" but rather files. I'd just simplify it to "Index all project files"

@b4n
Copy link
Member

b4n commented Feb 17, 2016

Otherwise, LGTM

gtk_misc_set_alignment(GTK_MISC(label), 0, 0);
e->generate_tag_prefs = gtk_combo_box_text_new();
gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(e->generate_tag_prefs), _("Auto (generate if less than 300 files)"));
gtk_combo_box_text_append_text(GTK_COMBO_BOX_TEXT(e->generate_tag_prefs), _("Auto (index if less than 300 files)"));
Copy link
Member

Choose a reason for hiding this comment

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

BTW, it says 300 here, but 500 in the docs.

@techee
Copy link
Member Author

techee commented Feb 18, 2016

@b4n Thanks, fixed. As the linguistic problem disappeared because I renamed the label, I guess the patch is OK now so merging.

techee added a commit that referenced this pull request Feb 18, 2016
projectorganizer: Use the word "symbol" instead of "tag"
@techee techee merged commit 47d07f6 into geany:master Feb 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants