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

Login dropdown #154

Merged
merged 3 commits into from
Jun 6, 2018
Merged

Login dropdown #154

merged 3 commits into from
Jun 6, 2018

Conversation

jaypal412
Copy link
Contributor

@jaypal412 jaypal412 commented Jun 6, 2018

Ticket:https://www.pivotaltracker.com/story/show/158024365

Ticket: https://www.pivotaltracker.com/story/show/158024172

Why?

Feature : Implement login drop down .
Feature: When there is no product or any UI list. Footer comes at the bottom of the screen.

Bug: Dropdown is not showing correct in device view
Bug: Footer is coming top when there is no content in the body

This change addresses the need by:

Made changes using the media query and the CSS required for the login dropdown

To achieve this I have made changes in the all body CSS and CSS of a footer.

[delivers #158024172]

[delivers #158024365]

@jaypal412 jaypal412 changed the title Login drond Login dropdown Jun 6, 2018
Copy link
Member

@gopalshimpi gopalshimpi left a comment

Choose a reason for hiding this comment

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

@jaypal412 Please go through requested changes asap.

</ul>
</div>
</div> -->

Copy link
Member

Choose a reason for hiding this comment

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

@jaypal412 why this code is commented?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Work in progress on that filter mobile menu

<accordion [closeOthers]="true">
<accordion-group>
<div accordion-heading class="clearfix">
I can have markup!
Copy link
Member

Choose a reason for hiding this comment

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

Can you please remove these hard coded values I can have markup! and many morre like this in this <accordion> tag.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Work in progress on that filter mobile menu

return this.showParrent ? 'show' : 'hide'
}

get stateName1() {
Copy link
Member

Choose a reason for hiding this comment

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

you can change this to proper variable name. stateName1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Work in progress on that filter mobile menu

this.store.select(getTaxonomies)
.subscribe(data => {
this.fillterList = data;
console.log(this.fillterList);
Copy link
Member

Choose a reason for hiding this comment

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

Remove this console log.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Work in progress on that filter mobile menu

@@ -24,7 +24,7 @@ <h4 *ngIf="categoryLevel.length > 0"> Catgeories</h4>
<li *ngFor="let levels of categoryLevel; let index =i" (click)="emitSelection()">
<a [routerLink]="['.']" class="list-group-item list-group-item-action" [queryParams]="{'q[name_cont]': levels.name, id: levels.id}">
<span class="maninname">
{{levels.name}}
{{levels.c}}
Copy link
Member

Choose a reason for hiding this comment

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

Why did you changed ? {{levels.name}} to {{level.c}}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This done by mistake. Change reverted.

}
get stateName() {
return this.showParrent ? 'show' : 'hide'
}

get stateName1() {
Copy link
Member

Choose a reason for hiding this comment

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

Change variable names.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

replaced with proper names

Copy link
Member

@pkrawat1 pkrawat1 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@pkrawat1 pkrawat1 merged commit 164ac98 into develop Jun 6, 2018
@pkrawat1 pkrawat1 deleted the logonpagemenu branch June 6, 2018 05:50
pkrawat1 pushed a commit that referenced this pull request Jun 13, 2018
**Feature** : Implement login drop down .
**Feature**: When there is no product or any UI list. Footer comes at the bottom of the screen. 

**Bug**: Dropdown is not showing correct in device view
**Bug**: Footer is coming top when there is no content in the body
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants