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

Issue #848 - Add style to cssfixme service #887

Merged
merged 1 commit into from
Dec 29, 2015
Merged

Conversation

magsout
Copy link
Member

@magsout magsout commented Dec 29, 2015

Before:

capture d ecran 2015-12-29 a 17 49 00

After:

capture d ecran 2015-12-29 a 17 40 19

We can improve it,it's a lick of paint

r? @karlcow @miketaylr @calexity @hallvors @tagawa

@magsout
Copy link
Member Author

magsout commented Dec 29, 2015

@miketaylr why I have to restart build each time ? if not, travis are not happy

@miketaylr
Copy link
Member

@miketaylr why I have to restart build each time ? if not, travis are not happy

I need to figure that out.... :( something weird about the search tests that fail every-other-time...

@miketaylr
Copy link
Member

r+, looks so much better!

miketaylr pushed a commit that referenced this pull request Dec 29, 2015
Issue #848 - Add style to cssfixme service
@miketaylr miketaylr merged commit 05b1e06 into master Dec 29, 2015
@miketaylr miketaylr deleted the style-cssfixme branch December 29, 2015 16:59
@karlcow
Copy link
Member

karlcow commented Dec 30, 2015

r+ with minor suggestions on the prose organization. Something along these lines

Let's put the big pile of text at the end of the page.
capture d ecran 2015-12-30 a 14 23 46

@magsout
Copy link
Member Author

magsout commented Dec 30, 2015

Nice @karlcow . What about output (résultat) ? Between Button and the text ?

@karlcow
Copy link
Member

karlcow commented Dec 30, 2015

Yes the output should come between the button and the text. Basically the text is always at the bottom.

@magsout
Copy link
Member Author

magsout commented Dec 30, 2015

ok @karlcow good.

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.

4 participants