Skip to content
This repository has been archived by the owner on Nov 14, 2018. It is now read-only.

Updated impress/css/style.css #2

Closed
wants to merge 7 commits into from
Closed

Updated impress/css/style.css #2

wants to merge 7 commits into from

Conversation

raghunayyar
Copy link
Contributor

Syntax error resolved.

raghunayyar and others added 7 commits August 27, 2012 22:26
Made the required UI changes, the messaging input also floats properly now. 
Bug Bug http://bugs.owncloud.org/thebuggenie/files/show/73 solved.
modernizr.js and HTML5 polyfills for placeholders added.
Syntax error solved.
Syntax Error resolved.
@DeepDiver1975
Copy link
Contributor

THX a lot.

Can you please remove .DS_Store?
And you included modernizer.js and modernizer.min.js - one is enough.

THX a lot

@DeepDiver1975
Copy link
Contributor

General question:
Shall we move modernizer.js to core/js/ in order to make it available for all apps?

@raghunayyar
Copy link
Contributor Author

Yeah, I too thought so.
Thomas (Tanghus) wanted me to put it in contacts.app to test how is it. I
was supposed to put it to the core/js/ folder anyhow. Should i do it?

On Thu, Aug 30, 2012 at 8:37 PM, Thomas Müller notifications@github.comwrote:

General question:
Shall we move modernizer.js to core/js/ in order to make it available for
all apps?


Reply to this email directly or view it on GitHubhttps://github.com//pull/2#issuecomment-8162405.

Sincerely,
Raghu Nayyar
Undergraduate,
B.E. Mechanical Engineering
BITS-Pilani, KK Birla Goa Campus
E-mail: me@iraghu.com

@DeepDiver1975
Copy link
Contributor

Sorry for the confusion - my question is targeting all other devs and more a future improvement.
Maybe nothing which was to be discussed here - please continue as you agreed with Thomas Tanghus.

@ghost ghost assigned karlitschek and tanghus Aug 31, 2012
LukasReschke added a commit that referenced this pull request Sep 5, 2012
Pull request #2 by @raghunayyar
@LukasReschke
Copy link
Contributor

I corrected the typo in the impress css file. (745590c)

Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants