Skip to content
This repository was archived by the owner on Nov 22, 2021. It is now read-only.

Add overlay attribute#815

Open
SamarRizvi wants to merge 5 commits intombenford:masterfrom
SamarRizvi:SamarRizvi-patch-overlay
Open

Add overlay attribute#815
SamarRizvi wants to merge 5 commits intombenford:masterfrom
SamarRizvi:SamarRizvi-patch-overlay

Conversation

@SamarRizvi
Copy link
Copy Markdown

Add overlay attribute to auto-complete directive so that if inside a container (like, for example, table) the suggestion list is outside the container and not inside. Also keep the auto-complete height to 0 as it gives unnecessary scroll when inside a container.

Add overlay attribute to auto-complete directive so that if inside a container (like, for example, table) the suggestion list is outside the container and not inside. Also keep the auto-complete height to 0 as it gives unnecessary scroll when inside a container.
Add necessary styling for overlay attribute.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.4%) to 99.407% when pulling 799235c on SamarRizvi:SamarRizvi-patch-overlay into 87d0e6b on mbenford:master.

@SamarRizvi SamarRizvi changed the title And overlay attribute Add overlay attribute Aug 11, 2017
Make ng-style less complex. Using function call instead.
Create overlayStyle object to use in ng-style in template
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.4%) to 99.407% when pulling 267898e on SamarRizvi:SamarRizvi-patch-overlay into 87d0e6b on mbenford:master.

Use overlayStyle object for ng-style that is set in auto-complete directive when overlay attribute is true.
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.6%) to 99.211% when pulling 916c371 on SamarRizvi:SamarRizvi-patch-overlay into 87d0e6b on mbenford:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.6%) to 99.211% when pulling 916c371 on SamarRizvi:SamarRizvi-patch-overlay into 87d0e6b on mbenford:master.

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.

2 participants