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

[AI Help] Question: can i use :empty pseudo elements in edge for android? #5

Open
2 of 3 tasks
develleoper opened this issue Jul 7, 2023 · 4 comments
Open
2 of 3 tasks

Comments

@develleoper
Copy link

What is the issue with the answer?

While Edge Android support is not listed on the MDN page for the :empty pseudo class, the response inaccurately claimed that the page gave no support information, which is false.

How would you categorize this issue?

  • The answer is harmful / unsafe.
  • The answer is not correct.
  • The answer is not helpful.

What did you ask?

can i use :empty pseudo elements in edge for android?

What did AI Help answer?

According to the provided MDN documentation, the :empty pseudo-class is not mentioned to be supported in any specific browser, including Edge for Android. Therefore, it is unclear whether Edge for Android supports the :empty pseudo-class. It is recommended to refer to the official documentation or perform testing to determine the browser support for the :empty pseudo-class in Edge for Android.

What MDN pages did AI Help consult?

@caugner
Copy link
Contributor

caugner commented Jul 7, 2023

Thank you @develleoper for taking the time to report this issue.

We're aware of this limitation regarding browser compatibility, and it is due to the fact that we currently only provide the Markdown from mdn/content as context to GPT, without rendering macros like the {{Compat}} one that adds the browser compatibility data in the form of a table.

In the short-term, we're looking into having AI Help refer to the Browser compatibility section for the corresponding feature in this case.

In the long-term, we're considering embedding information about browser support of a given feature into the context, so that the question can be answered directly.

@develleoper
Copy link
Author

This might merit being its own issue but until answers are fully reliable, presenting the feature without an explicit, visible, up-front caveat holds the potential to mislead newer folks who don't understand the nuance of LLM's.

@caugner
Copy link
Contributor

caugner commented Jul 8, 2023

presenting the feature without an explicit, visible, up-front caveat

I'm pleased to let you know that we have decided to add a more prominent disclaimer that will land on production soon, and which should address this valid concern that has been raised by you and other community members. 🎉

As for the issue about browser compatibility questions, let me be transparent about the fact that we identified that the blog post mentions "queries about APIs, browser compatibility, or coding best practices", and we will be updating it shortly to avoid setting this wrong expectation, until AI Help can actually answer questions about browser compatibility. We will indicate that the blog post was changed by an "Updated" note at the bottom of its content.

Again, I want to thank you for your contributions, and for using the "Report an issue with this answer" link. We will update this issue when we change the way AI Help answers questions involving browser compatibility. Stay tuned, and keep reporting other issues you may experience! 🙏

@caugner
Copy link
Contributor

caugner commented Dec 4, 2023

@develleoper Thanks again for reporting this issue. 👍

As we're working on an iteration, I would like to share with you what AI Help 2.0 might answer.

Can you please check how the following answer performs with regard to the issue you reported above? 🙏

can i use :empty pseudo elements in edge for android?

Yes, the :empty pseudo-class is supported in Microsoft Edge for Android. According to the MDN documentation on :empty, Edge has full support for this pseudo-class since version 12, which was released on July 28, 2015. Since Edge for Android is based on Chromium, it inherits support for most of the web standards that Chrome for Android supports.

For more details, you can refer to the MDN documentation on the :empty pseudo-class here.

@caugner caugner added the review label Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants