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

CRM-21406 - Add export cases action #87

Merged
merged 1 commit into from
Nov 13, 2017
Merged

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Nov 8, 2017

This depends on civicrm/civicrm-core#11254

@guanhuan @mukeshcompucorp there is a css bug in ShorDitch which is causing this to not display properly. The circled text is missing, (I manipulated css in my console to show it). The offending line is

.crm-container.ui-dialog .ui-dialog-content h3 {
  display: none;
}

I consider that to be a very bad css rule. I would advise whoever wrote it to go back to the drawing board and find a different solution to whatever problem they were trying to solve.

exportcases

@jamienovick
Copy link

jamienovick commented Nov 8, 2017

@guanhuan @mukeshcompucorp I'd like to know who is creating lines of CSS display:none in Shoreditch. Not really acceptable?!

And who reviewed the PR and approved it?

@colemanw
Copy link
Member Author

colemanw commented Nov 8, 2017

display:none is fine to use if it's done in a targeted way. My problem with this rule is it's lack of specificity. It hides all h3 elements in every dialog without regard to context or content.

@mukeshcompucorp
Copy link
Contributor

@jamienovick , I found out this one, it's from civicrm/org.civicrm.shoreditch@9375492
This seems a blind move but, as it was an year old css we did not had backstopjs and I think no body noticed such thing. only thing in mind would be some code is moved to other file.

But, again I agree this one is bad code and should be handled. Purpose of this code should be to hide only title (h3) from popup which are under public vacancy list /civicrm/vacancy/publiclisting?reset=1

PS: scss/civicrm/vacancies/_vacancies.scss should be part of CiviHR so, as per current status that file does not exist in shoreditch

I'm adding this in my note and will tackle it while working on CiviHR tickets as it's barely 2-4 minute of work.

@colemanw colemanw merged commit 5256720 into civicrm:master Nov 13, 2017
@colemanw colemanw deleted the export branch November 17, 2017 16:50
swastikpareek pushed a commit to swastikpareek/org.civicrm.civicase that referenced this pull request Oct 22, 2019
…ctions

C51-116: Activity Bulk Action - Delete Feature
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.

3 participants