Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

Fix #13274 - make Brackets margin/padding remoteHighlight work like Chrome one #13297

Merged
merged 2 commits into from
Apr 24, 2017

Conversation

Worie
Copy link
Contributor

@Worie Worie commented Apr 10, 2017

Added support for transform-origin and fixed the way of showing border in highlight div.

This contains a little bit more of changes than the previous one and probably will need a bit of refactor, but it should work. Please mention me if it isn't working for some sets of properties and provide the piece of code that does not look correct.

I've tested/developed it on OSX Sierra 10.12.4, Chrome 57.0.2987.133 (64-bit)

@saurabh95
Copy link
Contributor

The issue which I mentioned in the last PR is resolved now.

@Worie
Copy link
Contributor Author

Worie commented Apr 19, 2017

@swmitra what do you think?

@swmitra
Copy link
Collaborator

swmitra commented Apr 19, 2017

It seems to be working fine 👍
Let me have a look at the change set ...

Copy link
Collaborator

@swmitra swmitra left a comment

Choose a reason for hiding this comment

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

Tested the feature manually. Working fine for different scenarios I have tested. Changeset seems good to me.

@swmitra
Copy link
Collaborator

swmitra commented Apr 24, 2017

Good work worie 👍
Merging the PR, this will give us more time to identify issues (if any) well before 1.10 release.

@swmitra swmitra merged commit 70901f2 into adobe:master Apr 24, 2017
@swmitra
Copy link
Collaborator

swmitra commented Apr 24, 2017

@Worie I am working on the same set of files for some other feature. I will refactor it in my PR 😄

@Worie Worie deleted the worie/13274-lp-highlight-transform branch April 24, 2017 08:12
@Worie
Copy link
Contributor Author

Worie commented Apr 24, 2017

Awesome! I'm glad it turned out ok, feel free to ping me if necessary :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants