Skip to content

feat: Hide non-visible traits #16

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.

Sign up for GitHub

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

Draft
SpeedyD wants to merge 2 commits into
base: extension/alt-trait-types
Choose a base branch
from
Draft

feat: Hide non-visible traits #16

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f10c97e
feat: Hide non-visible traits
SpeedyD Dec 29, 2022
370b56a
fix: Undoing my over-engineering
SpeedyD Dec 29, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions app/Services/FeatureService.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -315,6 +315,7 @@ public function updateFeature($feature, $data, $user, $parent = null)
'display_separate' => $alt ? (isset($data['alt']['display_separate'][$key]) ? 1 : 0) : 1,
'image' => isset($data['alt']['image'][$key]) ? $data['alt']['image'][$key] : null,
'remove_image' => $alt ? (isset($data['alt']['remove_image'][$key]) ? 1 : 0) : 0,
'is_visible' => $alt ? (isset($data['alt']['is_visible'][$key]) ? 1 : 0) : 0,
];

// If the ID is already set, modify the existing feature
Expand Down
4 changes: 4 additions & 0 deletions resources/views/admin/features/create_edit_feature.blade.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -150,6 +150,10 @@
{!! Form::checkbox('alt[display_separate]['.$altType->id.']', 1, $altType->display_separate, ['class' => 'form-check-input', 'data-toggle' => 'toggle']) !!}
{!! Form::label('alt[display_separate]', 'Display Separately', ['class' => 'form-check-label ml-3']) !!} {!! add_help('If enabled, this trait will be displayed separately from its parent in general trait listings, including species\' visual trait indexes if enabled.') !!}
</div>
<div class="form-group">
{!! Form::checkbox('alt[is_visible]['.$altType->id.']', 1, $altType->id ? $altType->is_visible : 1, ['class' => 'form-check-input', 'data-toggle' => 'toggle']) !!}
{!! Form::label('alt[is_visible]['.$altType->id.']', 'Is Visible', ['class' => 'form-check-label ml-3']) !!} {!! add_help('If turned off, the trait will not be visible in the trait list or available for selection in search and design updates. Permissioned staff will still be able to add them to characters, however.') !!}
</div>
@if(isset($altType->display_separate) && $altType->display_separate)
<h4>Preview</h4>
<hr/>
Expand Down
6 changes: 3 additions & 3 deletions resources/views/world/_feature_entry.blade.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
@if($feature->feature_category_id)
<div><strong>Category:</strong> {!! $feature->category->displayName !!}</div>
@endif
@if($feature->parent_id)
@if($feature->parent_id && $feature->parent->is_visible)
<div><strong>Parent Trait:</strong> {!! $feature->parent->displayName !!}</div>
@endif
@if($feature->species_id)
Expand All @@ -16,14 +16,14 @@
<div class="world-entry-text parsed-text">
{!! $feature->parsed_description !!}
</div>
@if($feature->altTypes->count())
@if($feature->altTypes()->visible()->get()->count())
<h5 class="inventory-header">
Alternate Types
<a class="small collapse-toggle collapsed" href="#alt-{{ $feature->id }}" data-toggle="collapse">Show</a></h3>
</h5>
<div class="collapse show" id="alt-{{ $feature->id }}">
<ul>
@foreach($feature->altTypes as $altType)
@foreach($feature->altTypes()->visible()->get() as $altType)
<li>
{!! $altType->displayName !!} <a href="{{ $altType->searchUrl }}" class="world-entry-search text-muted"><i class="fas fa-search"></i></a>
@if(!$altType->display_separately)
Expand Down