Skip to content

feat(material): add prototype tabs component. - #1977

Closed
jelbourn wants to merge 1 commit into
angular:masterfrom
jelbourn:md-tabs
Closed

jelbourn wants to merge 1 commit into
angular:masterfrom
jelbourn:md-tabs

Conversation

@jelbourn

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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
Copy Markdown
Contributor 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

rkirov commented May 22, 2015

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

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

@rkirov

rkirov commented May 26, 2015

Copy link
Copy Markdown
Contributor

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

@jelbourn

Copy link
Copy Markdown
Contributor Author

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

@tbosch

tbosch commented Jun 1, 2015

Copy link
Copy Markdown
Contributor

Blocked on dart-archive/ts2dart#173

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

vsavkin commented Jun 8, 2015

Copy link
Copy Markdown
Contributor

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

@jelbourn

jelbourn commented Jun 8, 2015

Copy link
Copy Markdown
Contributor Author

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

@mprobst

mprobst commented Jun 12, 2015

Copy link
Copy Markdown
Contributor

The bump has happened.

@vsavkin

vsavkin commented Jun 15, 2015

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
Contributor Author

Now blocked on #2627

@tbosch

tbosch commented Jun 29, 2015

Copy link
Copy Markdown
Contributor

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

@jelbourn

Copy link
Copy Markdown
Contributor 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

mhevery commented Aug 4, 2015

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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