Skip to content

fix(aio): swap feature headers for divs to remove anchors#17156

Merged
vicb merged 1 commit into
angular:masterfrom
sjtrimble:features-sans-headers
Jun 1, 2017
Merged

fix(aio): swap feature headers for divs to remove anchors#17156
vicb merged 1 commit into
angular:masterfrom
sjtrimble:features-sans-headers

Conversation

@sjtrimble

@sjtrimble sjtrimble commented Jun 1, 2017

Copy link
Copy Markdown
Contributor

Closes #16824

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x")

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)
Anchors appear on headers when hovered upon.

What is the new behavior?
Headers have been substituted with divs and appropriate class and style which no longer show anchors when hovered upon.

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information: N/A

@sjtrimble sjtrimble added comp: aio action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 1, 2017
@mary-poppins

Copy link
Copy Markdown

The angular.io preview for 4d1b02d9b0d0bdb62385b043b341cf477189c055 is available here.

@sjtrimble sjtrimble force-pushed the features-sans-headers branch from 4d1b02d to 3558e7e Compare June 1, 2017 06:10
@mary-poppins

Copy link
Copy Markdown

The angular.io preview for 3558e7e is available here.

}

.feature-section img {
max-width: 70px;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think there needs to be some clean up wrt img styling. There seem to be rules scattered all over this file 😃
But not directly related to this PR.

@gkalpak gkalpak added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 1, 2017
@vicb vicb merged commit 47e4fca into angular:master Jun 1, 2017
asnowwolf pushed a commit to asnowwolf/angular that referenced this pull request Aug 11, 2017
juleskremer pushed a commit to juleskremer/angular that referenced this pull request Aug 28, 2017
@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 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

aio: no anchor tags on the temporary marketing home page.

5 participants