_("Creating dummy text with Geany"),

• "0.4.4",

• VERSION,

so you follow GP versioning now?

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472/files/a129fc448e74739c1b5fd7...

• config_dir_old = g_strconcat(geany->app->configdir, G_DIR_SEPARATOR_S,

• "plugins", G_DIR_SEPARATOR_S, "geanylipsum", NULL);
  

• config_file_old = g_strconcat(config_dir_old, G_DIR_SEPARATOR_S,

• "lipsum.conf", NULL);
  

• config_dir = g_strconcat(geany->app->configdir, G_DIR_SEPARATOR_S,

• "plugins", G_DIR_SEPARATOR_S, "lipsum", NULL);
  

• if (g_file_test(config_file_old, G_FILE_TEST_EXISTS))
• {

• if (dialogs_show_question(
  

• 	_("Renamed plugin detected!\n"
  

• 	  "\n"
  

• 	  "GeanyLipsum has been renamed to Lipsum -- you surely have "
  

• 	  "already recognised it. \n"
  

• 	  "Geany is able to migrate your old plugin configuration by "
  

• 	  "moving the old configuration file to new location.\n"
  

• 	  "Attention: this will not include your keybindings.\n"
  

should probably read "Warning:", shouldn't it?

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472/files/a129fc448e74739c1b5fd7...

Otherwise LGBI

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472#issuecomment-238106575

• {

• if (dialogs_show_question(
  

• 	_("Renamed plugin detected!\n"
  

• 	  "\n"
  

• 	  "GeanyLipsum has been renamed to Lipsum -- you surely have "
  

• 	  "already recognised it. \n"
  

• 	  "Geany is able to migrate your old plugin configuration by "
  

• 	  "moving the old configuration file to new location.\n"
  

• 	  "Attention: this will not include your keybindings.\n"
  

• 	  "Move now?")))
  

• {
  

• 	if (g_rename(config_dir_old, config_dir) == 0)
  

• 	{
  

• 		dialogs_show_msgbox(GTK_MESSAGE_INFO,
  

• 			_("Your configuration directory has been "
  

• 			  "successfully moved from \"%s\" to \"%s\"."),
  

For translatable strings the %s must be order independent in case translation changes the order, use `n$` to specify the parameter: `Your configuration directory has been successfully moved from "%1$s" to "%2$s".`

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472/files/a129fc448e74739c1b5fd7...

• {

• if (dialogs_show_question(
  

• 	_("Renamed plugin detected!\n"
  

• 	  "\n"
  

• 	  "GeanyLipsum has been renamed to Lipsum -- you surely have "
  

• 	  "already recognised it. \n"
  

• 	  "Geany is able to migrate your old plugin configuration by "
  

• 	  "moving the old configuration file to new location.\n"
  

• 	  "Attention: this will not include your keybindings.\n"
  

• 	  "Move now?")))
  

• {
  

• 	if (g_rename(config_dir_old, config_dir) == 0)
  

• 	{
  

• 		dialogs_show_msgbox(GTK_MESSAGE_INFO,
  

• 			_("Your configuration directory has been "
  

• 			  "successfully moved from \"%s\" to \"%s\"."),
  

Do translators know about `%n$`?

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472/files/a129fc448e74739c1b5fd7...

_("Creating dummy text with Geany"),

• "0.4.4",

• VERSION,

Yes. Doesn't make sense to have own numbering schema here

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472/files/a129fc448e74739c1b5fd7...

config_file = g_strconcat(geany->app->configdir, G_DIR_SEPARATOR_S, "plugins", G_DIR_SEPARATOR_S, "geanylipsum", G_DIR_SEPARATOR_S, "lipsum.conf", NULL);

• #ifndef G_OS_WIN32
• /* We try only to move if we are on not Windows platform */
• config_dir_old = g_build_filename(geany->app->configdir,

• "plugins", "geanylipsum", NULL);
  

• config_file_old = g_build_filename(config_dir_old,

• "lipsum.conf", NULL);
  

• config_dir = g_build_filename(geany->app->configdir,

• "plugins", "lipsum", NULL);
  

• if (g_file_test(config_file_old, G_FILE_TEST_EXISTS))
• {

• if (dialogs_show_question(
  

• 	_("Renamed plugin detected!\n"
  

• 	  "\n"
  

• 	  "GeanyLipsum has been renamed to Lipsum -- as you will "
  

• 	  "already recognised it. \n"
  

*as you will **have** already recognised.* Add *have* and remove *it*. https://github.com/geany/geany-plugins/pull/472#discussion_r73810656

(though I'd rather say *"as you will already have realized"*, but whatever @elextr says is likely correct)

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472/files/7f33eb6f5962ad92cbc1e9...

config_file = g_strconcat(geany->app->configdir, G_DIR_SEPARATOR_S, "plugins", G_DIR_SEPARATOR_S, "geanylipsum", G_DIR_SEPARATOR_S, "lipsum.conf", NULL);

• #ifndef G_OS_WIN32
• /* We try only to move if we are on not Windows platform */
• config_dir_old = g_build_filename(geany->app->configdir,

• "plugins", "geanylipsum", NULL);
  

• config_file_old = g_build_filename(config_dir_old,

• "lipsum.conf", NULL);
  

• config_dir = g_build_filename(geany->app->configdir,

• "plugins", "lipsum", NULL);
  

• if (g_file_test(config_file_old, G_FILE_TEST_EXISTS))
• {

• if (dialogs_show_question(
  

• 	_("Renamed plugin detected!\n"
  

• 	  "\n"
  

• 	  "GeanyLipsum has been renamed to Lipsum -- as you will "
  

• 	  "already recognised it. \n"
  

"As you may have already noticed, GeanyLipsum has been renamed to just Lipsum."

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472/files/7f33eb6f5962ad92cbc1e9...

config_file = g_strconcat(geany->app->configdir, G_DIR_SEPARATOR_S, "plugins", G_DIR_SEPARATOR_S, "geanylipsum", G_DIR_SEPARATOR_S, "lipsum.conf", NULL);

• #ifndef G_OS_WIN32
• /* We try only to move if we are on not Windows platform */
• config_dir_old = g_build_filename(geany->app->configdir,

• "plugins", "geanylipsum", NULL);
  

• config_file_old = g_build_filename(config_dir_old,

• "lipsum.conf", NULL);
  

• config_dir = g_build_filename(geany->app->configdir,

• "plugins", "lipsum", NULL);
  

• if (g_file_test(config_file_old, G_FILE_TEST_EXISTS))
• {

• if (dialogs_show_question(
  

• 	_("Renamed plugin detected!\n"
  

• 	  "\n"
  

• 	  "GeanyLipsum has been renamed to Lipsum -- as you will "
  

• 	  "already recognised it. \n"
  

"As you may have already noticed, GeanyLipsum has been renamed to just Lipsum."

even better

Not sure the additional sentence is needed as suggesting to move the configuration seems clear enough to me, but why not.

--- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/geany/geany-plugins/pull/472/files/7f33eb6f5962ad92cbc1e9...