entrouvert
/
gadjo
15
0
Fork 0
Code Issues Pull Requests 2 Releases Activity

scss: remove useless button focus declaration (#77614) #25

Open
tjund wants to merge 1 commits from wip/77614-scss-button-remove-focus-condition into main
pull from: wip/77614-scss-button-remove-focus-condition
merge into: entrouvert:main
Conversation 3 Commits 1 Files Changed 1 +2 -6
tjund commented 2023-05-15 16:10:00 +02:00
Owner
Copy Link

$button-background doesn't exist

$button-background doesn't exist
tjund added 1 commit 2023-05-15 16:10:00 +02:00
gitea/gadjo/pipeline/head This commit looks good Details
ea20dc3cdb scss: remove useless button focus declaration (#77614) 
$button-background doesn't exist
fpeters commented 2023-05-15 16:19:30 +02:00
Owner
Copy Link

L'idée était d'avoir du code au maximum proche de publik-base-theme pour faciliter la recopie de changements.

L'idée était d'avoir du code au maximum proche de publik-base-theme pour faciliter la recopie de changements.
tjund commented 2023-05-15 18:11:41 +02:00
Author
Owner
Copy Link

Ok (mais avec #77630 on s'éloigne de toute façon, un moment, de PBT).

Ok (mais avec #77630 on s'éloigne de toute façon, un moment, de PBT).
fpeters commented 2023-05-15 22:14:25 +02:00
Owner
Copy Link

Ok (mais avec #77630 on s'éloigne de toute façon, un moment, de PBT).

À minimiser quand même, en imaginant qu'on avance aussi, à un moment, vers les variables CSS dans publik-base-theme.

(édit : mais je répondais surtout à "qui me semble inutile" qui était donné dans le ticket, pour expliquer l'utilité que ça avait, on traite ce ticket comme on l'entend).

> Ok (mais avec #77630 on s'éloigne de toute façon, un moment, de PBT). À minimiser quand même, en imaginant qu'on avance aussi, à un moment, vers les variables CSS dans publik-base-theme. (édit : mais je répondais surtout à "qui me semble inutile" qui était donné dans le ticket, pour expliquer l'utilité que ça avait, on traite ce ticket comme on l'entend).
fpeters requested changes 2023-05-30 16:11:44 +02:00
fpeters left a comment
Owner
Copy Link

(à rebaser)

(à rebaser)
All checks were successful
gitea/gadjo/pipeline/head This commit looks good
Details
This pull request has changes conflicting with the target branch.
  • gadjo/static/css/_forms.scss
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b wip/77614-scss-button-remove-focus-condition main
git pull origin wip/77614-scss-button-remove-focus-condition

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff wip/77614-scss-button-remove-focus-condition
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/gadjo#25
No description provided.
Delete Branch "wip/77614-scss-button-remove-focus-condition"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?