Skip to content

Conversation

@jelbourn
Copy link
Member

Screenshot:
Screenshot of tabs

This is very early and doesn't do any of the fancy animation or styling stuff.

FYI @IgorMinar if you're curious how it came out after our brief paring.

Review on Reviewable

Copy link
Contributor

Choose a reason for hiding this comment

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

What if the removed tab was selected

Copy link
Member Author

Choose a reason for hiding this comment

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

Changed to do something more sensible.

@rkirov rkirov added the @lgtm label May 22, 2015
@rkirov
Copy link
Contributor

rkirov commented May 22, 2015

Abstract Pair(ProtoViewRef, ElementRef) (pick a better name) from the tabs logic and merge.

Copy link
Contributor

Choose a reason for hiding this comment

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

add a todo for the failure modes:

  • md-tab without label
  • md-tab with md-tab-label but no md-tab-content

Since this is a WIP, doesn't need to be handled in this PR.

@jelbourn
Copy link
Member Author

Converted to TS and added interface. All other comments addressed. Waiting for fix in ts2dart for interfaces to move forward.

@rkirov
Copy link
Contributor

rkirov commented May 26, 2015

Did you try @implements annotation from lang? It might already work.

@jelbourn
Copy link
Member Author

Martin says the fix for ts2dart is very simple so I'm going to wait for that.

@tbosch
Copy link
Contributor

tbosch commented Jun 1, 2015

Blocked on dart-archive/ts2dart#173

@tbosch tbosch assigned jelbourn and unassigned mhevery Jun 1, 2015
@vsavkin
Copy link
Contributor

vsavkin commented Jun 8, 2015

@jelbourn dart-archive/ts2dart#173 has been closed. Are you still blocked?

@jelbourn
Copy link
Member Author

jelbourn commented Jun 8, 2015

Was waiting on @mprobst to do a ts2dart release + version bump, I'll check on this tomorrow.

@mprobst
Copy link
Contributor

mprobst commented Jun 12, 2015

The bump has happened.

@vsavkin
Copy link
Contributor

vsavkin commented Jun 15, 2015

@jelbourn you should be good to go!

@vsavkin vsavkin added action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed state: blocked labels Jun 15, 2015
@jelbourn
Copy link
Member Author

Now blocked on #2627

@tbosch
Copy link
Contributor

tbosch commented Jun 29, 2015

@jelbourn What is the status here? Render services are ready to go...

@jelbourn
Copy link
Member Author

I've been working on other things since #2627 was fixed. I'll try to fix this up and get it submitted this week.

@mhevery
Copy link
Contributor

mhevery commented Aug 4, 2015

Cleaning up PRs. This one has not been touched for a while. Please create new PR once ready.

@mhevery mhevery closed this Aug 4, 2015
@jelbourn jelbourn deleted the md-tabs branch May 24, 2016 01:13
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants