-
Notifications
You must be signed in to change notification settings - Fork 8
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
profile display labels issue and carousel issue dots activate on scroll fixed #277
profile display labels issue and carousel issue dots activate on scroll fixed #277
Conversation
src/assets/i18n/en.json
Outdated
"VOICE_SEARCH_FAILED":"Voice search failed, please try again" | ||
"VOICE_SEARCH_FAILED":"Voice search failed, please try again", | ||
"ADD_PHOTO":"ADD PHOTO", | ||
"REMOVE_PHOTO":"REMOVE PHOTO" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it Remove photo or Remove Current Photo?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it should be like remove photo and if we keep as remove current photo the hindi language is not meaningful for that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep the same as previous one, else this might come up as an issue and we need products confirmation too
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay
@@ -1,6 +1,5 @@ | |||
<div class="side-navigation"> | |||
<div class="side-nav-header"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed the block as add the style instead of keeping listing from the top
@@ -61,6 +61,7 @@ export class ProfilePage { | |||
"preferred_language":this.selectedLanguage | |||
}).subscribe((res:any)=>{ | |||
this.setLanguage(res.result.preferred_language); | |||
this.loadFormAndData(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is this added here?
Any reason?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when we are changing the language in that page after changing we need to load the form again with translated stuff
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
these changes are for the carousal dots active UI which was dicussed morning?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes
No description provided.