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

Summary field looked out of place. #489

Closed
wants to merge 1 commit into from

Conversation

MichaelJJames
Copy link

@MichaelJJames MichaelJJames commented Nov 14, 2017

Summary field wasn't formatted correctly

gtfzo8k

This css will make it look like

gtfzqkq

Summary field wasnt formated correctly 

https://pasteboard.co/GTFzo8K.png

this css will make it look like 

https://pasteboard.co/GTFzQkq.png
@robbieaverill
Copy link
Contributor

There is some relevant discussion at #485

@robbieaverill
Copy link
Contributor

We should check that this issue still exists with silverstripe/silverstripe-admin#446 merge into core

@dhensby
Copy link
Contributor

dhensby commented Feb 27, 2018

Now merged

@robbieaverill
Copy link
Contributor

Ok, this PR could still do with being merged for now I think. I still think this issue should be resolved in framework, but we can patch for now anyway.

Here's what it looks like for me on SS 4.1 with blog 3.0.x:

image

@MichaelJJames would you like to rebase this PR?

MichaelJJames added a commit to MichaelJJames/silverstripe-blog that referenced this pull request Apr 5, 2018
@MichaelJJames
Copy link
Author

#525

@robbieaverill
Copy link
Contributor

Let's follow up conversation on #525

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants