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

Marketplace: Improve the UI #857

Merged

Conversation

zspencer
Copy link
Member

@zspencer zspencer commented Oct 2, 2022

Screen Shot 2022-10-02 at 12 00 39 PM

Screen Shot 2022-10-02 at 12 00 45 PM

Screen Shot 2022-10-02 at 12 01 01 PM

Screen Shot 2022-10-02 at 12 01 08 PM

dependabot bot and others added 4 commits October 1, 2022 00:16
Co-authored-by: Neer Thapa <nirmalathapa@users.noreply.github.com>
Co-authored-by: Neer Thapa <nirmalathapa@users.noreply.github.com>
This makes the user experience for adding a product even tighter; by
hiding the Product form until the Distributor is adding a new product.

Once they do, we replace the entire marketplace with the New Product
form; and upon successful create, we replace the entire marketplace with
the full Marketplace view!

Co-authored-by: Neer Thapa <nirmalathapa@users.noreply.github.com>
@zspencer zspencer requested a review from anaulin October 2, 2022 19:04
dependabot bot and others added 4 commits October 5, 2022 23:53
Add alt text to spotlight

Tried to add alt text as attribute but isn't working. plz help.

* Spotlight: Use `"attr"` rather than `:attr` when setting Alt Text

So, it turns out there's a pretty nasty wart in the `furniture`
implementation where setting the data requires you to know the type of
the key being set in the dictionary.

That's pretty gross, and probably something we should fix at the
architectural level, since it's cost us a ton of brain-space already and
we're only just writing our first few bits of Furniture.

That said, in the meantime as you add additional attributes to the Image
item, we should be able to follow the pattern of using string keys.

Co-authored-by: Ana Ulin <anaulin@users.noreply.github.com>
Co-authored-by: Zee Spencer <zspencer@users.noreply.github.com>
Still need to figure out how to redirect to furniture

Co-authored-by:  Kelly <KellyAH@users.noreply.github.com>
Co-authored-by:  Zee <zspencer@users.noreply.github.com>
Co-authored-by:  Ana <anaulin@users.noreply.github.com>
@sadiejay sadiejay merged commit 0b362f8 into ensemble/marketplace-furniture Oct 6, 2022
@sadiejay sadiejay deleted the zs/ensemble/marketplace/improve-ui branch October 6, 2022 00:38
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