Skip to content

Conversation

@jancborchardt
Copy link
Member

@jancborchardt jancborchardt commented Oct 19, 2018

Just like @Espina2 did on the website and also suggested for the app some time ago, so all credit to him! 🎉

This works very well with our form language, with the logo, and especially with the upcoming font change to Nunito: #11932

screenshot from 2018-10-19 18-55-55
screenshot from 2018-10-19 18-55-31

screenshot from 2018-10-19 16-12-11
buttons apps
screenshot from 2018-10-19 16-07-09
(That’s all the screenshots for now cause I ran into a maintenance mode I can’t recover from …)

Please review @nextcloud/designers and let me know if you find any non-pilled buttons.

@MariusBluem
Copy link
Member

Am a bit sceptic about this, since text boxes are not in this pill design ...

@skjnldsv
Copy link
Member

There is some css magic to do for the rounded white borders (see https://user-images.githubusercontent.com/925062/47232684-a3d36a80-d3d0-11e8-9cc9-f6807d545c56.png) It looks not ok with the antialiasing :)

Otherwise I really like it. ❤️

Though it will be indeed a bit odd with the inputs.
@jancborchardt can you provide a screenshot of an input[text] joined to a submit.icon-confirm? :)

@jancborchardt
Copy link
Member Author

Am a bit sceptic about this, since text boxes are not in this pill design ...

@MariusBluem yeah, I was also wondering about this. The reason I didn’t do the inputs yet is because traditionally only the search field is rounded like that. I would keep the inputs as they are for now, we can still adjust. (But even on our website I find it odd that the input fields are pill-style.)

There is some css magic to do for the rounded white borders (see https://user-images.githubusercontent.com/925062/47232684-a3d36a80-d3d0-11e8-9cc9-f6807d545c56.png) It looks not ok with the antialiasing :)

Yeah – I think a big factor of this is cause that specific screenshot was made with 150% zoom. It looks better normally.

Though it will be indeed a bit odd with the inputs.
@jancborchardt can you provide a screenshot of an input[text] joined to a submit.icon-confirm? :)

Sorry, my installation is busted atm because of a maintenance mode issue, can’t do more screenshots right now. :\ (But for the joined inputs nothing should change – the icon-confirm should be floating on the right of the input, without background so you don’t even see it’s round.)

@skjnldsv
Copy link
Member

Sorry, my installation is busted atm because of a maintenance mode issue, can’t do more screenshots right now. :\ (But for the joined inputs nothing should change – the icon-confirm should be floating on the right of the input, without background so you don’t even see it’s round.)

Yes, but the input also have a focus and active state, so this is not exactly like that :)
Let's see later how it looks and how we can improve if needed ;)

@jancborchardt
Copy link
Member Author

@skjnldsv @ChristophWurst so, could you approve? :)

Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

The inline input + confirm has still the proper design :)
Let's get this in!
capture d ecran_2018-10-23_12-38-04

@skjnldsv
Copy link
Member

Rebased!

@danxuliu
Copy link
Member

The inline input + confirm has still the proper design :)

Not everywhere, although this may be a problem specific to the share authentication page:
round-button-share-auth

@skjnldsv
Copy link
Member

@danxuliu nice catch!! :)
I'm guessing something from the guest css?

Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

@jancborchardt
Copy link
Member Author

@danxuliu @skjnldsv fixed :)
screenshot from 2018-10-23 15-01-03

@jancborchardt
Copy link
Member Author

Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
@MorrisJobke
Copy link
Member

JSUnit failure is due to codecov issue.

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

Labels

3. to review Waiting for reviews design Design, UI, UX, etc. enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants