Add code snippets style #290

Closed
anthonydillon opened this Issue Jan 26, 2016 · 17 comments

Comments

Projects
None yet
7 participants
Owner

anthonydillon commented Jan 26, 2016

Currently the logic behind the background colour of code snippets is a lighter version of the brand color. This works for Ubuntu as it gives us the correct light orange but in the case of Canonical it makes the code snippets light pink.

Should we sent the background to a set colour or if there something clever we can do with the brand colour in all situations?

We have a new code style now on Ubuntu and Cloud which have a purple background. Should we use that?

Contributor

yaili commented Feb 19, 2016

Yes, let's use the new style.

@yaili yaili assigned anthonydillon and unassigned yaili Feb 19, 2016

@yaili yaili changed the title from Design of code snippets to Add code snippets style Mar 7, 2016

Contributor

yaili commented Mar 7, 2016

I've found two different styles for the default pre tag:
screen shot 2016-02-19 at 14 16 34
screen shot 2016-02-19 at 14 17 08

I'm assuming the black version is the latest one, so we should incorporate that in Vanilla. /cc @Lucapaulina

Owner

anthonydillon commented Mar 7, 2016

This is the latest code snippet design:
get started juju

Contributor

richmccartney commented Mar 7, 2016

I think they should all be reviewed. We don't have a definitive code block style as well. We need to add the copy style to them and maybe code line numbers as well.

The black style is the latest one for code. We use the example Ant linked
for short snippets but not code blocks.

On Mon, 7 Mar 2016, 14:55 Richard McCartney, notifications@github.com
wrote:

I think they should all be reviewed. We don't have a definitive code block
style as well. We need to add the copy style to them and maybe code line
numbers as well.


Reply to this email directly or view it on GitHub
ubuntudesign#290 (comment)
.

Contributor

richmccartney commented Mar 7, 2016

Yeah on MAAS we have large code blocks for YAML and XML output. These all have code line numbers similar to GigHub

Contributor

yaili commented May 5, 2016

Agreed at off-site:

  • .code: < code > element (Vanilla)
  • .code-block: white/light with numbers (Vanilla)
  • .code-block--terminal: dark (Vanilla)
  • .code-snippet: white/light with copy-to-clipboard (Vanilla)
  • .code-snippet--terminal: dark with copy-to-clipboard (Vanilla)

@yaili yaili added enhancement and removed design labels May 5, 2016

Contributor

yaili commented May 9, 2016

@anthonydillon Has this been done?

Owner

anthonydillon commented May 9, 2016

No, the code snippets just have mono font at the moment.

On Mon, 9 May 2016 at 14:16 Inayaili de León Persson <
notifications@github.com> wrote:

@anthonydillon https://github.com/anthonydillon Has this been done?


You are receiving this because you were mentioned.

Reply to this email directly or view it on GitHub
ubuntudesign#290 (comment)

Contributor

yaili commented May 9, 2016

@anthonydillon You're assigned to this bug. Are you working on this, or should it be re/de-assigned?

Owner

anthonydillon commented May 9, 2016

Unassign please. Not sure why I was assigned.

On Mon, 9 May 2016 at 15:31 Inayaili de León Persson <
notifications@github.com> wrote:

@anthonydillon https://github.com/anthonydillon You're assigned to this
bug. Are you working on this, or should it be re/de-assigned?


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
ubuntudesign#290 (comment)

@yaili yaili referenced this issue in canonical-webteam/snapcraft.io-static-pages May 19, 2016

Closed

Visual design #2

@barrymcgee barrymcgee removed the enhancement label Jun 1, 2016

dholbach commented Jun 2, 2016

This would be super important to us in the Community team, as we'd like to adopt this for developer.ubuntu.com.

dholbach commented Jun 2, 2016

Are you going to support different programming languages and markup in the <code> blocks?

Contributor

deadlight commented Jun 2, 2016

That's definitely something to aim for but, initially, we just need to standardise our current usage.

dholbach commented Jun 2, 2016

I just learned from @caldav that we're using a different piece of code for programming languages, etc. Still having this bug fixed would help us a lot in a future move to Vanilla.

Owner

barrymcgee commented Jun 13, 2016

Fixed by #370

@barrymcgee barrymcgee closed this Jun 13, 2016

Nice one!

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