search

OfficeDev / microsoft-teams-library-js

JavaScript library for use by Microsoft Teams apps
https://docs.microsoft.com/microsoftteams/platform/
Other
430 stars 199 forks source link

Add event for tab change #2435

Open sschoeb opened 3 months ago

sschoeb commented 3 months ago

With the latest release on Teams for android the tabs do not reload when the user navigates between these.

As with the reload the data was always up-to-date we did not have to do anything here. But with this (in my opinion breaking) change, we now have to implement a logic that ensures the tab's data is refreshed whenever the user navigates to it.

Two questions on this:

  1. Is this "no-page-reload"-behaviour of teams mobile going to be the new standard and will also appear on the other clients (windows, ...)?
  2. What would be the proposed solution to be notified when the user opens a tab he had opened already before?
Nivedipa-MSFT commented 3 months ago

@sschoeb - Thanks for reporting your issue. We are checking this internally, we will update you soon.

Nivedipa-MSFT commented 3 months ago

@sschoeb - We received an update from the engineering team stating that the 'no-page-reload' behavior of Teams mobile will not be available on other platforms and is only applicable to mobile clients. Additionally, you can perform a pull-to-refresh action to reload the page.

Please let us know if you have any further query here?

sschoeb commented 3 months ago

Hi @Nivedipa-MSFT - thanks for checking.. but you answrt does not help us at all:

  1. Pull-to-refresh is currebtly not working on teams for android.
  2. What is the suggested solution?
  3. How should we know about this kind of breaking change and why is there no announcement of it? This change just broke our UI without any prior warning from your side. Quize sure we are not the only one...
sschoeb commented 2 months ago

@Nivedipa-MSFT is there any update on this? Our users are confused as since this change obviously the data on the tab might be outdated.

sschoeb commented 1 month ago

@Nivedipa-MSFT Would be great to have an update on this. At least a statement on what we have to expect on this tabs. Please answer the questions above: https://github.com/OfficeDev/microsoft-teams-library-js/issues/2435#issuecomment-2271595699

Nivedipa-MSFT commented 1 month ago

@sschoeb - Apologies for the delay. We are checking this with engineering team, we will let you know once we get any updates on it.

Nivedipa-MSFT commented 2 weeks ago

@sschoeb - We received a reply from the engineering team stating that the pull-to-refresh feature does not work on Android due to bugs and limitations in Android WebView. The solution provided by the engineering team is to add a Refresh Menu item under the overflow menu in the top right corner. Using this, we can reload the page.

Please let us know if you have any further query here.

sschoeb commented 2 weeks ago

@Nivedipa-MSFT thank you for your feedback. I do have a few questions about this "solution" and its usability:

  1. How should the user know about this refresh menu item? (and do they think that the provided solution is a good solution?
  2. How is it a good solution to show outdated data to the user and force him to refresh manually?
  3. How is the release process of you guys so that in the future we would know about this kind of breaking changes?
  4. What is the idea of the engineering team to have different behaviour depending on the platform without giving us a chance to distinguish these platforms?

I numbered the questions so that you can answer each one without missing a point. Thank you!

Nivedipa-MSFT commented 2 weeks ago

@sschoeb - We are checking this with engineering team, we will let you know once we get any updates on it.