General tidy-up of front end code/removal of debug elements

@jessebaker marked the MR as ready, I'm assuming that means this needs review.

P.S.: Next time, please mark the issue as needing review 🙏

The branch is called csstidyup, but the issue title is "general tidy up". Can we update the issue title to be a bit more precise? 🙏

I'm surprised this is passing tests, because .findByText() does not seem to be used correctly and we're not updating ui/package.json despite importing something new?

After that's addressed, please assign this to @bnjmnm for review — the change in the .module file needs his sign-off.

Actually … @bnjmnm is out today, but I'm pretty sure @hooroomoo knows enough about the JSX theme engine stuff to be able to review that one change in the .module file 🤓

🙈 😅 — my bad, sorry!

@jessebaker: can you address @hooroomoo's https://git.drupalcode.org/project/experience_builder/-/merge_requests/1...? 🙏

After that, please assign this to @bnjmnm, he should confirm my theoretical answer to @hooroomoo's excellent review question.

🏓 One last round of feedback from me, trying to catch everything that @bnjmnm might ask.

Nice to see this happening. I had a suggestion for applying the formatting more broadly to the tests, and once that is OK I'll probably click approve.

Didn't know @bnjmnm wrote experience_builder_preprocess_form_element(), so then this is definitely ready to go as far as I'm concerned 😊 🚢

I meant to mark this RTBC in #15 😅🙈

P.S.: I think it might be easier to first land 📌 Document Cypress test best practices (especially how to write reliable tests) Needs review — that touches the Cypress tests in a much bigger way, and this issue's MR will likely conflict, but is mostly doing trivial whitespace reformatting … which is easier to redo. So, @jessebaker, I suggest you land that issue first and then land this one 🤞

Needs final sign-off from either @bnjmnm or @hooroomoo, assigning to @bnjmnm since he already reviewed it previously.

Good call on doing a round of cleanup when it's still a manageable scope.

Automatically closed - issue fixed for 2 weeks with no activity.