Skip to content

Conversation

@BlackPhlox
Copy link

@BlackPhlox BlackPhlox commented Dec 7, 2023

Successor of #23, solves #22 & #24 & (#20 & #26)

#22 Background Theme

@boyko-ant, I've updated it so that the user's selected theme is now used in the background of the widget.

#24 Non-ASCII character handling

I also updated to handling wiki pages with UTF-8 characters which were based on the changes made by Mikael Ekelund in his fork: stratiteq@e34430e.

However, the change did not work as all the encoding was done after the " " to "+", which meant that all spaces got replaced by %2B instead of + as per DevOps implementation. This PR is confirmed to now also work for UTF-8 characters, such as Ü.

#20 & #26 Flickering widget when typing

Due to having a listener on onFieldChanged, means that whenever any field on the work item, the widget gets reloaded, which creates the undeliverable flickering, that listener is now removed and no longer flickers when typing in any text field.

Other remarks

Also updated categories as Plan and track is not a valid category anymore.

Once these fixes as been pushed, I would highly recommend that @stratiteq and I (@BlackPhlox) take down our own published extensions of our forks, or at least make a deprecation notice, referencing the original extension as so to prevent confusion for other DevOps users.

Regarding the issue of linking to multiple wikis (#25), it should probably be documented in the description of the extension, that this is a limitation of DevOps and you have to link from the wiki page to work-item instead as shown by this comment: #25 (comment)

@boyko-ant
Copy link
Member

Thanks for your contribution @BlackPhlox
I will try to review it as soon as possible and publish the new version.

P.S. Sorry for not being able to answer faster. We, as Ukrainians, are still having issues with our noisy neighbors now.

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

Successfully merging this pull request may close these issues.

2 participants