Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

Why doesn't cursor change to pointer/hand on Tabs, Pagination, Dropdown Toggle etc in angular-ui-bootstrap? #475

Closed
sgon00 opened this issue May 26, 2013 · 5 comments

Comments

@sgon00
Copy link

sgon00 commented May 26, 2013

I have to add the following code:

<style type="text/css">a {cursor:pointer;}</style>

to make cursor change to pointer/hand on links for Tabs, Pagination, Dropdown Toggle etc.. in angular-ui-bootstrap.

Why doesn't it change to pointer by default? Is this done intentionally? this is not the normal case in pure bootstrap.

Thanks a lot.

@bekos
Copy link
Contributor

bekos commented May 26, 2013

You can read about it in #174.

@pkozlowski-opensource
Copy link
Member

@sgon00 as commented by @bekos we can't add href attributes to links in a template as it seems to be causing reload issues for some people.

Closing for now as I don't see how we could handle this better, I'm afraid :-( Feel free to reopen if you've got a proposal.

@sgon00
Copy link
Author

sgon00 commented May 29, 2013

OK, Can you please confirm the following is right or not?
(1) for <a> tag in angularjs-ui-bootrap, (especially with ng-click attribute), I should never use "href" attribute?
(2) if I want to link to some url, I should use $location.url() instead of "href" ?
(3) for the cursor problem, simply add "a {cursor:pointer;}" as a workaround, this is fine?
I am a newbie on angularjs, please let me know if my understanding is corrent or not.
Thanks.

@pkozlowski-opensource
Copy link
Member

@sgon00 some answers below:

(1) It is not specific to this project, empty href is not really needed if you just use it as a markup with ngClick. See http://docs.angularjs.org/api/ng.directive:a for more.

(2) No, if you've got real URL you should rather use href or ng-href.

(3) Yes, this is more bootstrap's CSS issue where presence (or not) of href results in different styling.

@innatewonderer
Copy link

Found this useful:
https://docs.angularjs.org/api/ng/directive/ngHref

By adding href: "" to my a tag, I was able to get cursor: pointer to show up.

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

No branches or pull requests

4 participants