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

feat: ✨ add config to hide moderation #393

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

Conversation

melMass
Copy link
Member

@melMass melMass commented Mar 3, 2024

fixes #392

wip, I will test it further later

Copy link

github-actions bot commented Mar 3, 2024

PR Preview Action v1.4.7
🚀 Deployed preview to https://teia-community.github.io/teia-ui/previews/pr-393/
on branch gh-pages at 2024-03-23 22:06 UTC

@Zir0h
Copy link
Contributor

Zir0h commented Mar 3, 2024

Nice! Works for me 👍 but maybe we can get rid of the not so subtle background too? 🤣
2024-03-03-104533_884x745_scrot

@Zir0h
Copy link
Contributor

Zir0h commented Mar 3, 2024

The "moderated" banner also seems to "win" from the transaction popup 😆
2024-03-03-161926_499x407_scrot

@melMass
Copy link
Member Author

melMass commented Mar 3, 2024

Ahaha I did not test it further and thought it was the last state I'll remove it!

About the Z index nice catch I'll double check the ranges and maybe define a few "plane" as variables (bg,fg, overlay)

@melMass
Copy link
Member Author

melMass commented Mar 23, 2024

I'm confused wasn't this deployed @Zir0h ?

@Zir0h
Copy link
Contributor

Zir0h commented Mar 23, 2024

I'm confused wasn't this deployed @Zir0h ?

the activation of the moderation banner was done yes
this PR gives the minter the option to disable the moderation styling in case they got the message and they will tag pieces accordingly in the future

@melMass
Copy link
Member Author

melMass commented Mar 23, 2024

I'm dumb I thought it was the original one! I need some sleep 😅 !

@melMass
Copy link
Member Author

melMass commented Mar 23, 2024

And 3 weeks old...? I lost notion of time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants