Skip to content

Add border on hover for code blocks#4259

Merged
ara4n merged 2 commits intoelement-hq:developfrom
ollieh:code-block-border
Jun 18, 2017
Merged

Add border on hover for code blocks#4259
ara4n merged 2 commits intoelement-hq:developfrom
ollieh:code-block-border

Conversation

@ollieh
Copy link
Copy Markdown
Contributor

@ollieh ollieh commented Jun 9, 2017

The border for non hovered codeblocks is the same colour as the background, and there so the content doesn't move when the border is added.

Signed-off-by: Oliver Hunt <oliver@hunt.bz>
}
.mx_EventTile_body pre {
position: relative;
border: 1px solid #f8f8f8;
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm, surely this should be a $ variable so that it themes correctly?

@ara4n
Copy link
Copy Markdown
Member

ara4n commented Jun 9, 2017

one comment, otherwise lgtm. Also, you might want to just consider 1px solid transparent for the null case

Signed-off-by: Oliver Hunt <oliver@hunt.bz>
@ara4n
Copy link
Copy Markdown
Member

ara4n commented Jun 18, 2017

thanks

@ara4n ara4n merged commit 4afec6f into element-hq:develop Jun 18, 2017
dbkr pushed a commit that referenced this pull request Jun 19, 2017
Add border on hover for code blocks
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.

2 participants