? ? Pending

User tests: Successful: Unsuccessful:

avatar kofaysi
kofaysi
24 Apr 2019

…and alias

Pull Request for Issue #24650.

Testing Instructions

Create a contact with a name/title of 200+ characters

Expected result

The title and the alias will wrap, long words will break.

Actual result

The title and the alias does not wrap and pushes other parameters (access, hits, language, ID) behind the right edge of the screen.

Documentation Changes Required

IDK about any.

avatar kofaysi kofaysi - open - 24 Apr 2019
avatar kofaysi kofaysi - change - 24 Apr 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 24 Apr 2019
Category Administration com_contact
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 24 Apr 2019

I have tested this item ? unsuccessfully on 2643f7a

Without PR

Screen Shot 2019-04-24 at 08 49 21

With PR

Screen Shot 2019-04-24 at 08 49 52


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/24712.
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 24 Apr 2019 - Tested unsuccessfully
avatar kofaysi
kofaysi - comment - 24 Apr 2019

a break-word class has to be added a line below the current change
<div class="pull-left break-word">
But I do not understand how can I change the existing PR. I can create a new PR, but is that the right approach?

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 24 Apr 2019

i don't know but pretts aure a dev will answer.

avatar hardik-codes
hardik-codes - comment - 24 Apr 2019

@kofaysi you can do the changes on your local machine and then push those changes. In this way a new commit will be added to the existing PR

If you are finding it difficult and if you want I can do a PR for you with your suggested changes

avatar kofaysi
kofaysi - comment - 24 Apr 2019

Unfortunately, I'm making changes and PRs through the web interface. Looking into the existing PR, I cannot make changes to the code.

So, please, if you feel so, make a change. Thank you!

avatar brianteeman
brianteeman - comment - 24 Apr 2019

Go to https://github.com/kofaysi/joomla-cms

Select the branch - in this case it is staging

Make the changes you want there and they will appear here

avatar hardik-codes
hardik-codes - comment - 24 Apr 2019

@kofaysi you can follow what @brianteeman suggested

avatar kofaysi kofaysi - change - 24 Apr 2019
Labels Added: ?
avatar kofaysi
kofaysi - comment - 24 Apr 2019

Thank you for the suggestion. Changes are made and pushed. (Well, that was very straightforward.)

avatar kofaysi kofaysi - change - 24 Apr 2019
The description was changed
avatar kofaysi kofaysi - edited - 24 Apr 2019
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 24 Apr 2019

I have tested this item successfully on 1314cc9


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

avatar franz-wohlkoenig franz-wohlkoenig - test_item - 24 Apr 2019 - Tested successfully
avatar brianteeman
brianteeman - comment - 24 Apr 2019

@kofaysi glad you worked it out.

avatar Quy
Quy - comment - 10 May 2019

I have tested this item successfully on 1314cc9


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

avatar Quy Quy - change - 10 May 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 10 May 2019

RTC


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

avatar Quy Quy - test_item - 10 May 2019 - Tested successfully
avatar SniperSister SniperSister - change - 16 May 2019
Labels Added: ?
avatar SniperSister SniperSister - change - 16 May 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-05-16 04:37:58
Closed_By SniperSister
avatar SniperSister SniperSister - close - 16 May 2019
avatar SniperSister SniperSister - merge - 16 May 2019
avatar SniperSister
SniperSister - comment - 16 May 2019

Merged, thank you for contribution, it's much appreciated!

Add a Comment

Login with GitHub to post a comment