Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Highlight the current tree #29

Closed
bjarnef opened this issue Mar 18, 2015 · 6 comments
Closed

Highlight the current tree #29

bjarnef opened this issue Mar 18, 2015 · 6 comments

Comments

@bjarnef
Copy link
Collaborator

bjarnef commented Mar 18, 2015

I think it would be great to highlight the current tree node :)

https://our.umbraco.org/forum/developers/extending-umbraco/63267-Custom-section-mark-currentNode-blue?p=0#comment213103

@warrenbuckley
Copy link
Member

@bjarnef if you do a pull request for this. I can't remember what me you & @abjerner got stuck on, as I thought we was close to a release? Do you remember?

@abjerner
Copy link
Member

At least for my part, we had some problems with the WebForms dialog due to the authentication bug in 7.2.2. I was pretty sure I added some code to handle that, but can't find it in the list of commits. I will look into whether I just forgot to commit or whatever I have been doing.

Anyways, according to Our, 7.2.3 is out tomorrow, so we might as well wait to see that everything works for that release.

@warrenbuckley
Copy link
Member

Yes makes sense to @abjerner to do a release to check it works with 7.2.1, 7.2.2 & 7.2.3

@abjerner
Copy link
Member

Oh well. Just needed to look in the developer branch: 9cbec98

@bjarnef
Copy link
Collaborator Author

bjarnef commented Mar 18, 2015

@warrenbuckley I can take a look at it tonight maybe or some of the next days..

as @abjerner mention there is the bug about UmbracoEnsuredPage in 7.2.2 .. so I think it will be great to wait for 7.2.3 to test in that version too and if the workaround 9cbec98 fix the issue for 7.2.2

then there is the $digest error #28 .. but I am not sure if it breaks anything?

@warrenbuckley
Copy link
Member

OK so if we can get these couple of things sorted perhaps this week or next then we could have a release (btw on holiday next week, so you guys may need to release it)

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

No branches or pull requests

3 participants