Skip to content

fix(list): add bottom border for last select item #12690

Closed
marcovtwout wants to merge 96 commits intoionic-team:masterfrom
marcovtwout:patch-1
Closed

fix(list): add bottom border for last select item #12690
marcovtwout wants to merge 96 commits intoionic-team:masterfrom
marcovtwout:patch-1

Conversation

@marcovtwout
Copy link

Fixes #12689

brandyscarney and others added 30 commits June 12, 2017 13:51
When `Tabs` are nested within each other, the highlight can get
misaligned. This prevents that by ensuring the affected
`.tab-highlight` is a direct child of the targeted `Tabs`.
…tabs

* wip

* wip

* progress

* wippy skippy

* getting there

* all tests passing except goBack

* unit tests pass again boi

* goBack tests pass

* great success

* the good stuff
danbucholtz and others added 22 commits July 14, 2017 11:55
…am#12386)

* chore(docs): add anchor links to documentation sub-sections

* chore(docs): make entire section headings clickable as anchors
* wip

* simple-nav, simple-tabs, simple-nav-then-tabs, simple-nested-navs all pass w00t w00t

* updates

* fix tests

* update test
* feat(generators): update templates

* feat(generators): add pipeName to templates
…stroying a nav controller while its transitioning
@AmitMY
Copy link
Contributor

AmitMY commented Aug 17, 2017

Is this an issue with ios and wp as well?

@marcovtwout
Copy link
Author

@AmitMY I don't think so (the original input css blames from #9679, which notes this is not an issue for ios/wp).

@brandyscarney brandyscarney changed the title Fixes #12689 fix(list): add bottom border for last select item Aug 31, 2017
@Ionitron
Copy link
Collaborator

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!

@Ionitron Ionitron closed this Mar 12, 2018
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.