Hey,

When you focus an input element on http://twitter.github.com/bootstrap/ . Then you will see the outline of that input field to nicely fade-in and when you deselect it fades out again. I really like this small detail, its much nicer on the eye then the instant outline color change normally done by browsers.

I propose we do this in the Seven theme too, we can probably go for the "Seven blue" outline color.

Comments

aspilicious’s picture

Status: Active » Needs review
StatusFileSize
new1.18 KB

Try this patch.

(this patch needs some minor adjustments but this is for testing)

Bojhan’s picture

Issue tags: +Usability

Awesome, looks like we might want to play some more with the color - for example going with #0074BD, that we use for links?

aspilicious’s picture

Issue tags: -Usability
StatusFileSize
new985 bytes

Second try, different colour

aspilicious’s picture

Issue tags: +Usability

Adding tag again

Bojhan’s picture

Issue tags: +Accessibility

Accessibility friends, can we do this? We are adding a outline: none;, but also creating a more beautiful outline.

aspilicious’s picture

Issue tags: -Accessibility
StatusFileSize
new1.2 KB

Another try but accessibility safe :)

Everett Zufelt’s picture

As long as there is still a perceivable outline, and it doesn't take too long to appear I don't see this being a problem. How long is the fade-in?

aspilicious’s picture

0.2 sec :)

mgifford’s picture

0.2s seems to be the fade in time. There are sites like this warning about outline:none http://outlinenone.com - but largely because they've been misapplied as you state.

I'll try to install this patch when I get chance. I think it will look sharp!

Bojhan’s picture

StatusFileSize
new18.84 KB
yoroy’s picture

Thanks for the quick screencast.

Overall I think this is a nice touch and worth doing.

The blue does seem a bit heavy overall. Additionally, by expanding the box around the form element by a few pixels, the spacing with the label above becomes a bit tight. Same for the 'Machine name' label to the right in this case.

Does the border get doubled from 1 to 2px? If so, I would first look at keeping that at 1px.
The other option is to reuse the current blue tint for focussed form elements in Seven: #aaccee

Bojhan’s picture

That does look better, it looks like its only the shadow thats a bit strong now.

yoroy’s picture

Status: Needs review » Needs work

Ok, needs work then

Bojhan’s picture

Status: Needs work » Needs review
StatusFileSize
new1.01 KB

Tried to incorperate yoroy's suggestions, it looks a lot cleaner now. Still really diggin it :) I thought this was already in core.

http://screencast.com/t/pLfBhpe6

yoroy’s picture

Status: Needs review » Reviewed & tested by the community

Yep, really nice.

dbazuin’s picture

The transistions is a little bit to fast. A transmission time of 0.25 is just enough for the effect we needed here.

Bojhan’s picture

I discussed with dbazuin and tried different approaches from 0.15, 0.25, 0.30, 0.35 to 0.45 and they didn't feel quite right to me. I suggest committing, this and returning to it - if after long time actual use we feel it needs additional tweaking.

catch’s picture

Status: Reviewed & tested by the community » Needs review

Do we still need with -webkit and -moz prefixes? It's my impression that recent releases removed the need for these at least in some cases, but I'm not able to find a definitive reference for what does and doesn't need a prefix now (if someone has one that'd be great).

catch’s picture

Status: Needs review » Fixed

aspilicious confirmed in irc that Safari 5 and firefox 3.6 both need it, so committed/pushed to 8.x.

yched’s picture

Nifty :-)

It does make the existing focus mark (orange, square, no fade effect) on the rest of the elements (checkboxes, radio, submit buttons, links...) look a bit off when tabbing through a form.

Bojhan’s picture

Oh, hmmpf - we should probably change that.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Bojhan’s picture

Status: Closed (fixed) » Needs work
lewisnyman’s picture

Status: Needs work » Postponed (maintainer needs more info)

I can't seem to recreate this, could we a screenshot?

Bojhan’s picture

Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (works as designed)

We removed this at some point, in all our refactoring. Lets revisit it later - as we evaluate how we can animate our UI more.