Skip to content

[4.0] add legend styles in settings#24012

Closed
bahl24 wants to merge 4 commits intojoomla:4.0-devfrom
bahl24:legend_format
Closed

[4.0] add legend styles in settings#24012
bahl24 wants to merge 4 commits intojoomla:4.0-devfrom
bahl24:legend_format

Conversation

@bahl24
Copy link
Contributor

@bahl24 bahl24 commented Feb 25, 2019

Pull Request for Issue - Legend styles which existed in J3 are missing, which makes navigation in settings difficult.

Expected result

screenshot from 2019-02-25 16-11-51
screenshot from 2019-02-25 16-10-47

Actual result

screenshot from 2019-02-25 16-18-06

@bahl24
Copy link
Contributor Author

bahl24 commented Feb 26, 2019

@infograf768 @C-Lodder What do you think?

@Razzo1987
Copy link
Contributor

I have tested this item 🔴 unsuccessfully on 5d18f97

no change :(


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

@wojsmol
Copy link
Contributor

wojsmol commented Feb 26, 2019

@Razzo1987 You mead to run npm run build:css after applying this patch - see Setting Up Your Local Environment.

@bahl24
Copy link
Contributor Author

bahl24 commented Feb 26, 2019 via email

@Razzo1987
Copy link
Contributor

I have tested this item ✅ successfully on 5d18f97

Sorry!

with npm run build:css works!


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

@ghost ghost added the J4 Issue label Apr 5, 2019
@ghost ghost removed the J4 Issue label Apr 13, 2019
@Quy
Copy link
Contributor

Quy commented Apr 14, 2019

Related PR #24322

@brianteeman
Copy link
Contributor

yes that PR fixes the problem which was a pr that should not have been merged

@ghost ghost changed the title [4.0]add legend styles in settings [4.0] add legend styles in settings Apr 15, 2019
@brianteeman
Copy link
Contributor

This should be closed as it was fixed with #24322

@bahl24 bahl24 closed this May 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants