? ? ? Pending

User tests: Successful: Unsuccessful:

avatar bahl24
bahl24
11 Feb 2019

Signed-off-by: Nitish Bahl nitishbahl24@gmail.com

Pull Request for Issue - 'insert image' should have capital first char

  1. New article->cms content

Expected result

screenshot from 2019-02-11 23-52-26

Actual result

screenshot from 2019-02-11 23-53-55

Should I add a language def which might help in translation in future

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
5.00

avatar bahl24 bahl24 - open - 11 Feb 2019
avatar bahl24 bahl24 - change - 11 Feb 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Feb 2019
Category Front End Plugins
avatar bahl24 bahl24 - change - 11 Feb 2019
Title
fixed language of 'insert image' to 'Insert Image'
[4.0]fixed language of 'insert image' to 'Insert Image'
avatar bahl24 bahl24 - edited - 11 Feb 2019
avatar Bakual
Bakual - comment - 11 Feb 2019

The comment behind it is actually the important part. That string should be translatable. So it should go through Text::_() and the string being defined in a language file.

avatar bahl24
bahl24 - comment - 11 Feb 2019

The comment behind it is actually the important part. That string should be translatable. So it should go through Text::_() and the string being defined in a language file.

Ok, that's what I meant to say by asking if we need a lang def.
@Bakual Done, kindly see if the string name is according to convention or needs to be modified

avatar bahl24 bahl24 - change - 11 Feb 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 11 Feb 2019
Category Front End Plugins Administration Language & Strings Front End Plugins
avatar Bakual
Bakual - comment - 11 Feb 2019

Done, kindly see if the string name is according to convention or needs to be modified

The name is fine, just change what Brian was already pointing out then this PR is perfect. Thanks already! ?

avatar bahl24 bahl24 - change - 12 Feb 2019
Labels Added: ?
avatar bahl24
bahl24 - comment - 12 Feb 2019

@brianteeman @Bakual @infograf768 Changes have been incorporated. Kindly test this

avatar Quy
Quy - comment - 13 Feb 2019

I have tested this item successfully on d6b074c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23878.

avatar Quy Quy - test_item - 13 Feb 2019 - Tested successfully
avatar infograf768
infograf768 - comment - 13 Feb 2019

I have tested this item successfully on ddd6014


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23878.

avatar infograf768
infograf768 - comment - 13 Feb 2019

I have tested this item successfully on ddd6014


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23878.

avatar infograf768 infograf768 - test_item - 13 Feb 2019 - Tested successfully
avatar infograf768 infograf768 - change - 13 Feb 2019
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 13 Feb 2019

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/23878.

avatar Bakual Bakual - change - 13 Feb 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-02-13 11:15:08
Closed_By Bakual
Labels Added: ?
avatar Bakual Bakual - close - 13 Feb 2019
avatar Bakual Bakual - merge - 13 Feb 2019
avatar Bakual
Bakual - comment - 13 Feb 2019

Merged, thanks for the contribution!

Add a Comment

Login with GitHub to post a comment