Skip to content
This repository was archived by the owner on Mar 4, 2025. It is now read-only.

SUP-2930, Register--> 'Create Password' text fileld showing one line at beginning. #648

Merged
merged 2 commits into from
Jan 13, 2016

Conversation

vikasrohit
Copy link
Contributor

@vic-appirio @nlitwin please let me know if you guys find any thing inappropriate here.

@vic-appirio I have seen in git history that you have made some changes (21cf847, 190ab97) to toggle password input styles about a month back. I am afraid if I have broken those changes. Please review my changes and please try it once on your local. It would really help me to be sure about the changes.

-- Fixed style of toggle with password directive to have smooth inner input box.

…at beginning.

-- Fixed style of toggle with password directive to have smooth inner input box.
@vic-tian
Copy link
Contributor

Nick took over the style in general. @nlitwin can you verify if everything is fine?

@vikasrohit
Copy link
Contributor Author

@nlitwin do you know if we need to override toggle-password and toggle-password-with-tips styles in https://github.com/appirio-tech/styles/blob/master/styles/topcoder/_tc-forms.scss? Can we remove that from styles repo?

…t Profile

-- Fixed padding of input after changes for SUP-2930. SUP-2930 fixed the wrapping of the label.
@vikasrohit
Copy link
Contributor Author

@nlitwin I have pushed changes for SUP-2970 as well in this PR because these two are closely related.

@nlitwin
Copy link
Contributor

nlitwin commented Jan 12, 2016

It looks like to me that we have CSS being applied to the toggle-password directives in the style guide (_tc-forms.scss) and locally (directives/toggle-password.scss), right? I think since this is a reusable component, we could move the local styles to the style guide. I've been adding new style guide updates to the branch "panels", until we move to webpack. Not sure if that answers your question.. haha :) Let me know if it doesn't!

@vikasrohit
Copy link
Contributor Author

Right now I have managed to fix it with changes in topcoder-app. May be we can move complete styles of toggle-password and toggle-password-with-tips to the styles repo later. Or let me know if you want me to move it to there now.

Merging the branch for now considering the v1.0.20 release tomorrow.

vikasrohit pushed a commit that referenced this pull request Jan 13, 2016
…h-pwd-style

SUP-2930, Register--> 'Create Password' text fileld showing one line at beginning.
@vikasrohit vikasrohit merged commit 42f1e79 into dev Jan 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants