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

Resolve Conflict with .container Class in LABJS #226

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Axthauvin
Copy link

Description

In LABJS, the .container class is used for the builder, potentially causing conflicts with HTML/CSS code.

This problem affects not just experienced programmers writing CSS code within LABJS, but also people with less knowledge who are generating code with tools like ChatGPT.

This pull request proposes a solution to address this conflict, enhancing user experience and code compatibility.

Example :

There are many ways to show the issue. The most event one is by asking a question to ChatGPT, where it can reply by using the .container class.

I queried ChatGPT with:

image
And it responded with:

image
image

Identifying why the .container class was overridden is not always easy, as it necessitates inspecting the console, which is not easily accessible for researchers, the primary users of the program.

Changes :

  1. Renamed the .container class to .labjs-container throughout the LABJS codebase to avoid conflicts.
  2. Updated the reference of the .container class in the index.html file to reflect the new name .labjs-container.

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.

None yet

1 participant