Skip to content

Conversation

mehedijaman
Copy link
Contributor

Summary

This PR introduces a placeholder prop to the AppDataSearch component, allowing consumers to define custom placeholder text instead of using the hardcoded "Search".

Changes

  • Added placeholder prop with default value "Search"
  • Updated template to use :placeholder="placeholder" instead of __('Search')
  • Maintains backward compatibility (defaults to "Search" if no prop is passed)

Usage Example

<AppDataSearch
    :url="route('student.index')"
    fields-to-search="name"
    placeholder="Search by student name"
></AppDataSearch>

### Summary
This PR introduces a `placeholder` prop to the `AppDataSearch` component, allowing consumers to define custom placeholder text instead of using the hardcoded "Search".

### Changes
- Added `placeholder` prop with default value `"Search"`
- Updated template to use `:placeholder="placeholder"` instead of `__('Search')`
- Maintains backward compatibility (defaults to `"Search"` if no prop is passed)

### Usage Example
```vue
<AppDataSearch
    :url="route('student.index')"
    fields-to-search="name"
    placeholder="Search by student name"
></AppDataSearch>
- Implemented a reusable multi-select combobox component (`AppMultiCombobox.vue`)
- Supports search, tag-style selected items, and keyboard navigation
- Emits `update:modelValue` with array of selected options
- Styled consistently with existing `AppCombobox` component
- Enables selecting multiple recipients in message compose form
…n State

Fullscreen Toggle

Added a fullscreen button with smooth rotate + scale animation.

Fullscreen state icon (maximize / minimize) persists in localStorage.

Automatically updates icon when exiting fullscreen via Esc.

Notes:

Fullscreen cannot be automatically restored on page reload due to browser security restrictions; only the icon state persists.

Logout and theme toggling functionality remain unaffected.
- Updated logout() to redirect to GET route 'adminAuth.loginForm' instead of POST login route.
- Ensures a fresh CSRF token is generated and avoids 419 Page Expired errors on first login after logout.
- Session is invalidated and CSRF token regenerated for security.
Fix Logout to Redirect to Login Form and Prevent 419 Error
Copy link
Collaborator

@daniel-cintra daniel-cintra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The logout system already works as designed/expected, considering the Laravel/Inertia flow... Please ensure eventual errors, are not related to local config/dev environment.

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