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

[4.0] add legend styles in settings #24012

Closed
wants to merge 4 commits into from
Closed

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