X Tutup
Skip to content

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

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

feat(material): add prototype tabs component.#1977
jelbourn wants to merge 1 commit intoangular: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
Copy link
Copy Markdown
Contributor

rkirov commented May 22, 2015

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

rkirov commented May 26, 2015

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

vsavkin commented Jun 8, 2015

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

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

mprobst commented Jun 12, 2015

The bump has happened.

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

Now blocked on #2627

@tbosch
Copy link
Copy Markdown
Contributor

tbosch commented Jun 29, 2015

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

X Tutup